@ -83,7 +83,8 @@ for API_FILE in ${API_FILES[*]}; do
API_CHANGE = ` git diff --name-only upstream/$BRANCH | grep " ${ API_FILE } " | grep -v "/CMakeLists.txt" || true `
if [ " ${ API_CHANGE } " ] && [ " ${ GIT_PR_ID } " != "" ] ; then
# NOTE: per_page=10000 should be ok for all cases, a PR review > 10000 is not human readable.
# approval_user_list: XiaoguangHu01 46782768,Xreki 12538138,luotao1 6836917,sneaxiy 32832641,qingqing01 7845005,guoshengCS 14105589,heavengate 12605721,kuke 3064195,Superjomn 328693,lanxianghit 47554610,cyj1986 39645414,hutuxian 11195205,frankwhzhang 20274488,nepeplwu 45024560,Dianhai 38231817,JiabinYang 22361972,chenwhql 22561442,zhiqiu 6888866,seiriosPlus 5442383,gongweibao 10721757,saxon-zh 2870059,Boyan-Liu 2870059, zhouwei25 52485244, Aurelius84 9301846, liym27 33742067, zhhsplendid 7913861.
# You can use http://caius.github.io/github_id/ to find Github user id.
# approval_user_list: XiaoguangHu01 46782768,Xreki 12538138,luotao1 6836917,sneaxiy 32832641,qingqing01 7845005,guoshengCS 14105589,heavengate 12605721,kuke 3064195,Superjomn 328693,lanxianghit 47554610,cyj1986 39645414,hutuxian 11195205,frankwhzhang 20274488,nepeplwu 45024560,Dianhai 38231817,chenwhql 22561442,zhiqiu 6888866,seiriosPlus 5442383,gongweibao 10721757,saxon-zh 2870059,Boyan-Liu 2870059, zhouwei25 52485244, Aurelius84 9301846, liym27 33742067, zhhsplendid 7913861, kolinwei 22165420, liuwei1031 46661762.
if [ " ${ API_FILE } " = = "paddle/fluid/op_use_default_grad_op_maker.spec" ] ; then
echo_line = "You must have one RD (sneaxiy (Recommend) or luotao1) approval for op_use_default_grad_op_maker.spec, which manages the grad_op memory optimization.\n"
check_approval 1 32832641 6836917
@ -123,9 +124,9 @@ fi
HAS_UNITTEST_SKIP = ` git diff -U0 upstream/$BRANCH | grep "^+[[:space:]]\{0,\}@unittest.skip" || true `
if [ " ${ HAS_UNITTEST_SKIP } " != "" ] && [ " ${ GIT_PR_ID } " != "" ] ; then
echo_line = " Unittest is not allowed to be disabled.\nYou must have one RD ( XiaoguangHu01, phlrain, luotao1, liuwei1031 or lanxianghit ) approval for the usage of @unittest.skip or @unittest.skipIf.\n${ HAS_UNITTEST_SKIP } \n "
check_approval 1 46782768 6836917 47554610 43953930 46661762
fi
echo_line = " Unittest is not allowed to be disabled.\nYou must have one RD ( kolinwei(Recommend), liuwei1031, or luotao1 ) approval for the usage of @unittest.skip or @unittest.skipIf.\n${ HAS_UNITTEST_SKIP } \n "
check_approval 1 22165420 6836917 46661762
fi
ALL_PADDLE_ENFORCE = ` git diff -U0 upstream/$BRANCH | grep "+" | grep -zoE "PADDLE_ENFORCE\(.[^,\);]+.[^;]*\);\s" || true `
if [ " ${ ALL_PADDLE_ENFORCE } " != "" ] && [ " ${ GIT_PR_ID } " != "" ] ; then