From 2d058cce52edbe72c855aabd90fb286da0890baf Mon Sep 17 00:00:00 2001 From: Chen Weihang Date: Wed, 2 Nov 2022 01:28:23 -0500 Subject: [PATCH] Add phi core file into ci checking list (#47564) * add phi core file into ci list, test=document_fix * remove repated file, test=document_fix --- tools/check_file_diff_approvals.sh | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/tools/check_file_diff_approvals.sh b/tools/check_file_diff_approvals.sh index d2419e7722..1ebe0ebb71 100644 --- a/tools/check_file_diff_approvals.sh +++ b/tools/check_file_diff_approvals.sh @@ -69,6 +69,18 @@ API_FILES=("CMakeLists.txt" "paddle/fluid/eager/autograd_meta.h" "paddle/fluid/eager/backward.cc" "paddle/fluid/eager/backward.h" + "paddle/phi/api/include/tensor.h" + "paddle/phi/core/tensor_base.h" + "paddle/phi/core/dense_tensor.h" + "paddle/phi/core/meta_tensor.h" + "paddle/phi/core/tensor_meta.h" + "paddle/phi/core/attribute.h" + "paddle/phi/core/device_context.h" + "paddle/phi/core/kernel_utils.h" + "paddle/phi/core/kernel_registry.h" + "paddle/phi/core/kernel_factory.h" + "paddle/phi/core/kernel_context.h" + "paddle/phi/core/infermeta_utils.h" ) approval_line=`curl -H "Authorization: token ${GITHUB_API_TOKEN}" https://api.github.com/repos/PaddlePaddle/Paddle/pulls/${GIT_PR_ID}/reviews?per_page=10000` @@ -191,6 +203,9 @@ for API_FILE in ${API_FILES[*]}; do elif [ "${API_FILE}" == "python/paddle/fluid/dygraph/layers.py" ] || [ "${API_FILE}" == "paddle/fluid/eager/grad_node_info.h" ] || [ "${API_FILE}" == "paddle/fluid/eager/grad_node_info.cc" ] || [ "${API_FILE}" == "paddle/fluid/eager/grad_tensor_holder.h" ] || [ "${API_FILE}" == "paddle/fluid/eager/grad_tensor_holder.cc" ] || [ "${API_FILE}" == "paddle/fluid/eager/tensor_wrapper.h" ] || [ "${API_FILE}" == "paddle/fluid/eager/autograd_meta.cc"] || [ "${API_FILE}" == "paddle/fluid/eager/autograd_meta.h"] || [ "${API_FILE}" == "paddle/fluid/eager/backward.cc"] || [ "${API_FILE}" == "paddle/fluid/eager/backward.h"]; then echo_line="You must have one RD (JiabinYang,chenwhql,phlrain) approval for ${API_FILE}, which manages the underlying code for PaddlePaddle.\n" check_approval JiabinYang chenwhql phlrain + elif [ "${API_FILE}" == "paddle/phi/api/include/tensor.h" ] || [ "${API_FILE}" == "paddle/phi/core/tensor_base.h" ] || [ "${API_FILE}" == "paddle/phi/core/dense_tensor.h" ] || [ "${API_FILE}" == "paddle/phi/core/meta_tensor.h" ] || [ "${API_FILE}" == "paddle/phi/core/tensor_meta.h" ] || [ "${API_FILE}" == "paddle/phi/core/attribute.h" ] || [ "${API_FILE}" == "paddle/phi/core/device_context.h" ] || [ "${API_FILE}" == "paddle/phi/core/kernel_utils.h" ] || [ "${API_FILE}" == "paddle/phi/core/kernel_registry.h" ] || [ "${API_FILE}" == "paddle/phi/core/kernel_factory.h" ] || [ "${API_FILE}" == "paddle/phi/core/kernel_context.h" ] || [ "${API_FILE}" == "paddle/phi/core/infermeta_utils.h" ]; then + echo_line="You must have one RD (chenwhql, phlrain, zyfncg, YuanRisheng) approval for changing ${API_FILE} , which manages the underlying code for PaddlePaddle PHI Library.\n" + check_approval chenwhql phlrain zyfncg YuanRisheng else echo_line="You must have one RD (XiaoguangHu01,chenwhql,zhiqiu,Xreki,luotao1,qili93) approval for ${API_FILE}, which manages the underlying code for fluid.\n" check_approval 1 46782768 12538138 6836917 22561442 6888866 16605440 -- GitLab