未验证 提交 7635af09 编写于 作者: A Aurelius84 提交者: GitHub

[NewIR]Add Approval checking for DEPS phi* in paddle/ir (#56714)

* [NewIR]Add Approval checking for DEPS phi* in paddle/ir

* fix conflict

* fix conflict

* fix branch

* fix cmd

* fix cmd

* fix cmd

* fix cmd

* test

* fix cmd

* test=document_fix

* add -r for xargs

* test=document_fix
上级 7adb4703
......@@ -230,6 +230,12 @@ for API_FILE in ${API_FILES[*]}; do
fi
done
DEPS_PHI_IN_IR=`git diff --name-only upstream/$BRANCH | grep -E "paddle/ir/" | grep "CMakeList" |xargs -r git diff -U0 upstream/$BRANCH --| grep "^\+" | grep "phi" || true`
echo "DEPS_PHI_IN_IR:${DEPS_PHI_IN_IR}"
if [ "${DEPS_PHI_IN_IR}" ] && [ "${DEPS_PHI_IN_IR}" != "" ]; then
echo_line="You must have one RD (Aurelius84, phlrain, zhangbo9674, winter-wang) approval for the CMakeLists.txt with DEPS phi* in paddle/ir directory.\n"
check_approval 1 Aurelius84 phlrain zhangbo9674 winter-wang
fi
FILTER=`git diff --name-only upstream/develop | grep -v "tools/"`
HAS_CONST_CAST=`git diff -U0 upstream/$BRANCH $FILTER | grep '^\+' | grep -o -m 1 "const_cast" || true`
if [ ${HAS_CONST_CAST} ] && [ "${GIT_PR_ID}" != "" ]; then
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册