未验证 提交 065608dd 编写于 作者: N Nyakku Shigure 提交者: GitHub

[CodeStyle] add more information when codestyle check failed (#47116)

上级 95ca886c
...@@ -30,6 +30,7 @@ if ! [[ $(python -V 2>&1 | awk '{print $2}' | awk -F '.' '{print $1$2}') -ge 36 ...@@ -30,6 +30,7 @@ if ! [[ $(python -V 2>&1 | awk '{print $2}' | awk -F '.' '{print $1$2}') -ge 36
fi fi
diff_files=$(git diff --numstat ${BRANCH} | awk '{print $NF}') diff_files=$(git diff --numstat ${BRANCH} | awk '{print $NF}')
num_diff_files=$(echo "$diff_files" | wc -l)
echo -e "diff files between pr and ${BRANCH}:\n${diff_files}" echo -e "diff files between pr and ${BRANCH}:\n${diff_files}"
echo "Checking code style by pre-commit ..." echo "Checking code style by pre-commit ..."
...@@ -37,18 +38,31 @@ pre-commit run --files ${diff_files};check_error=$? ...@@ -37,18 +38,31 @@ pre-commit run --files ${diff_files};check_error=$?
if test ! -z "$(git diff)"; then if test ! -z "$(git diff)"; then
echo -e '\n************************************************************************************' echo -e '\n************************************************************************************'
echo -e "These files have been formated by code format hook. You should use pre-commit to \ echo -e "These files have been formatted by code format hook. You should use pre-commit to \
format them before git push." format them before git push."
echo -e '************************************************************************************\n' echo -e '************************************************************************************\n'
git diff 2>&1 git diff 2>&1
fi fi
echo -e '\n***********************************' echo -e '\n************************************************************************************'
if [ ${check_error} != 0 ];then if [ ${check_error} != 0 ];then
echo "Your PR code style check failed." echo "Your PR code style check failed."
echo "Please install pre-commit locally and set up git hook scripts:"
echo ""
echo " pip install pre-commit==2.17.0"
echo " pre-commit install"
echo ""
if [[ $num_diff_files -le 100 ]];then
echo "Then, run pre-commit to check codestyle issues in your PR:"
echo ""
echo " pre-commit run --files" $(echo ${diff_files} | tr "\n" " ")
echo ""
fi
echo "For more information, please refer to our codestyle check guide:"
echo "https://www.paddlepaddle.org.cn/documentation/docs/zh/develop/dev_guides/git_guides/codestyle_check_guide_cn.html"
else else
echo "Your PR code style check passed." echo "Your PR code style check passed."
fi fi
echo -e '***********************************\n' echo -e '************************************************************************************\n'
exit ${check_error} exit ${check_error}
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册