Skip to content

Commit

Permalink
Add paddle_build.sh Check;test=document_fix (#57702)
Browse files Browse the repository at this point in the history
  • Loading branch information
tianshuo78520a authored Sep 25, 2023
1 parent 27476e7 commit 589f0f2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion paddle/scripts/paddle_build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2740,7 +2740,7 @@ function enable_unused_var_check() {
export FLAGS_enable_unused_var_check=1
}
function check_coverage_added_ut() {
# NOTE(risemeup1):The steps of checkinge added test can be placed on the cpu machine to save gpu resources
# NOTE(risemeup1):The step of checking added test can be placed on the cpu machine to save gpu resources
bash $PADDLE_ROOT/tools/check_added_ut.sh
}
function gen_doc_lib() {
Expand Down
4 changes: 4 additions & 0 deletions tools/check_file_diff_approvals.sh
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ API_FILES=("CMakeLists.txt"
"python/paddle/incubate/autograd/primitives.py"
"python/paddle/autograd/ir_backward.py"
"python/paddle/autograd/backward_utils.py"
"paddle/scripts/paddle_build.sh"
)

approval_line=`curl -H "Authorization: token ${GITHUB_API_TOKEN}" https://api.github.com/repos/PaddlePaddle/Paddle/pulls/${GIT_PR_ID}/reviews?per_page=10000`
Expand Down Expand Up @@ -223,6 +224,9 @@ for API_FILE in ${API_FILES[*]}; do
elif [ "${API_FILE}" == "python/paddle/autograd/ir_backward.py" ] || [ "${API_FILE}" == "python/paddle/autograd/backward_utils.py" ]; then
echo_line="You must be approved by Aurelius84(zhangliujie) or cxxly(chenxiaoxu) or xiaoguoguo626807(wangruting) or changeyoung98(chenzhiyang) for python/paddle/autograd/ir_backward.py or python/paddle/autograd/backward_utils.py changes.\n"
check_approval 1 Aurelius84 cxxly xiaoguoguo626807 changeyoung98
elif [ "${API_FILE}" == "paddle/scripts/paddle_build.sh" ]; then
echo_line="You must have one RD (tianshuo78520a or risemeup1 or zhangbo9674 or XieYunshen) for ${API_FILE} changes, which manages the Paddle CI on Linux.\n "
check_approval 1 tianshuo78520a risemeup1 zhangbo9674 XieYunshen
else
echo_line="You must have one RD (XiaoguangHu01,chenwhql,zhiqiu,Xreki,luotao1,qili93,Aurelius84) approval for ${API_FILE}, which manages the underlying code for fluid.\n"
check_approval 1 XiaoguangHu01 chenwhql zhiqiu Xreki luotao1 qili93 Aurelius84
Expand Down

0 comments on commit 589f0f2

Please sign in to comment.