split ci to multiple workflows

Author: SigureMo

.github/workflows/check-bypass.yml

@@ -0,0 +1,51 @@
+on:
+  workflow_call:
+    inputs:
+      workflow-name:
+        required: true
+        type: string
+    secrets:
+      github-token:
+        required: true
+    outputs:
+      can-skip:
+        description: "Whether the workflow can be skipped."
+        value: ${{ jobs.check-bypass.outputs.can-skip }}
+
+jobs:
+  check-bypass:
+    name: Check bypass
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    env:
+      CI_TEAM_MEMBERS: '["tianshuo78520a", "swgu98", "risemeup1", "XieYunshen", "luotao1"]'
+    outputs:
+      can-skip: ${{ steps.check-bypass.outputs.can-skip }}
+    steps:
+      - name: Cleanup
+        run: |
+          rm -rf * .[^.]*
+
+      - id: check-bypass
+        name: Check Bypass
+        uses: PFCCLab/ci-bypass@v2
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          non-pull-request-event-strategy: "never-skipped"
+          type: "composite"
+          composite-rule: |
+            {
+              "any": [
+                {
+                  "type": "labeled",
+                  "label": ["skip-ci: ${{ inputs.workflow-name }}", "skip-ci: all"],
+                  "username": ${{ env.CI_TEAM_MEMBERS }}
+                },
+                {
+                  "type": "commented",
+                  "comment-pattern": [".*/skip-ci ${{ inputs.workflow-name }}.*", ".*/skip-ci all.*"],
+                  "username": ${{ env.CI_TEAM_MEMBERS }}
+                }
+              ]
+            }

.github/workflows/codestyle-check.yml

@@ -0,0 +1,55 @@
+name: Codestyle Check
+
+on:
+  pull_request:
+    branches: ["develop"]
+
+jobs:
+  check-bypass:
+    name: Check bypass
+    if: ${{ inputs.can-skip != 'true' }}
+    uses: ./.github/workflows/check-bypass.yml
+    with:
+      workflow-name: "codestyle"
+    secrets:
+      github-token: ${{ secrets.GITHUB_TOKEN }}
+
+  pre-commit:
+    name: Pre Commit
+    needs: check-bypass
+    if: ${{ github.repository_owner == 'PaddlePaddle' && needs.check-bypass.outputs.can-skip != 'true' }}
+    runs-on: ubuntu-latest
+    env:
+      PR_ID: ${{ github.event.pull_request.number }}
+      BRANCH: develop
+
+    steps:
+      - name: Cleanup
+        run: |
+          rm -rf * .[^.]*
+
+      - name: Checkout base repo
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.base.ref }}
+          fetch-depth: 200
+
+      - name: Merge PR to test branch
+        run: |
+          git fetch origin pull/${PR_ID}/merge
+          git checkout -b test FETCH_HEAD
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v6
+        with:
+          python-version: "3.13"
+
+      - name: Install prek
+        run: |
+          uv tool install prek
+
+      - name: Run prek
+        run: |
+          set +e
+          bash -x ci_scripts/check_code.sh;EXCODE=$?
+          exit $EXCODE

ci_scripts/check_code.sh

@@ -22,21 +22,16 @@ set +x
 SCRIPT_DIR="$( cd "$( dirname "$0" )" && pwd )"
 cd $SCRIPT_DIR/..
 
-# use pre-commit 2.17
-if ! [[ $(pre-commit --version) == *"2.17.0"* ]]; then
-    pip install pre-commit==2.17.0 1>nul
-fi
-
 diff_files=$(git diff --name-only --diff-filter=ACMR ${BRANCH})
 num_diff_files=$(echo "$diff_files" | wc -l)
 echo -e "diff files between pr and ${BRANCH}:\n${diff_files}"
 
-echo "Checking code style by pre-commit ..."
-pre-commit run --files ${diff_files};check_error=$?
+echo "Checking code style by prek ..."
+prek run --files ${diff_files};check_error=$?
 
 if test ! -z "$(git diff)"; then
     echo -e '\n************************************************************************************'
-    echo -e "These files have been formatted by code format hook. You should use pre-commit to \
+    echo -e "These files have been formatted by code format hook. You should use prek to \
 format them before git push."
     echo -e '************************************************************************************\n'
     git diff 2>&1
@@ -45,15 +40,15 @@ fi
 echo -e '\n************************************************************************************'
 if [ ${check_error} != 0 ];then
     echo "Your PR code style check failed."
-    echo "Please install pre-commit locally and set up git hook scripts:"
+    echo "Please install prek locally and set up git hook scripts:"
     echo ""
-    echo "    pip install pre-commit==2.17.0"
-    echo "    pre-commit install"
+    echo "    pip install prek"
+    echo "    prek install"
     echo ""
     if [[ $num_diff_files -le 100 ]];then
-        echo "Then, run pre-commit to check codestyle issues in your PR:"
+        echo "Then, run prek to check codestyle issues in your PR:"
         echo ""
-        echo "    pre-commit run --files" $(echo ${diff_files} | tr "\n" " ")
+        echo "    prek run --files" $(echo ${diff_files} | tr "\n" " ")
         echo ""
     fi
     echo "For more information, please refer to our codestyle check guide:"

ci_scripts/ci_start.sh

@@ -102,17 +102,11 @@ else
 fi
 
 EXIT_CODE=0
-# 3 check code style/format.
-ls ${DIR_PATH}
-/bin/bash -x  ${DIR_PATH}/check_code.sh
-if [ $? -ne 0 ];then
-    EXIT_CODE=1
-fi
 
 git merge --no-edit upstream/${BRANCH}
 need_check_cn_doc_files=$(find_all_cn_api_files_modified_by_pr)
 echo $need_check_cn_doc_files
-# 4 Chinese api docs check
+# 3 Chinese api docs check
 if [ "${need_check_cn_doc_files}" = "" ] ; then
     echo "chinese api doc fileslist is empty, skip check."
 else
@@ -127,7 +121,7 @@ else
     fi
 fi
 
-# 5 Chinese api_label check
+# 4 Chinese api_label check
 /bin/bash -x ${DIR_PATH}/check_api_label_cn.sh
 if [ $? -ne 0 ];then
     set +x
@@ -141,13 +135,13 @@ fi
 if [ ${EXIT_CODE} -ne 0 ]; then
     set +x
     echo "=========================================================================================="
-    echo "Code style check or API Chinese doc check failed! Please check the error info above carefully."
+    echo "API Chinese doc check failed! Please check the error info above carefully."
     echo "=========================================================================================="
     set -x
     exit ${EXIT_CODE}
 fi
 
-# 6 Approval check
+# 5 Approval check
 /bin/bash  ${DIR_PATH}/checkapproval.sh
 if [ $? -ne 0 ];then
     exit 1