[GitHub] [seatunnel] FlechazoW commented on a diff in pull request #5515: [CI] Fix file change not be check when CI run in fork repository

["FlechazoW (via GitHub)" <gi...@apache.org>]

FlechazoW commented on code in PR #5515:
URL: https://github.com/apache/seatunnel/pull/5515#discussion_r1330334695


##########
.github/workflows/backend.yml:
##########
@@ -80,48 +80,61 @@ jobs:
       ut-modules: ${{ steps.ut-modules.outputs.modules }}
       it-modules: ${{ steps.it-modules.outputs.modules }}
     steps:
-      - uses: actions/checkout@v3 # required for push event
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: '2000'
+      - name: checkout apache seatunnel dev branch
+        id: git_init
+        run: |
+          /usr/bin/git remote add apache https://github.com/apache/seatunnel
+          /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --no-recurse-submodules --depth=2000 apache +refs/heads/dev*:refs/remotes/apache/dev* +refs/tags/dev*:refs/tags/dev*
+          /usr/bin/git checkout apache/dev
+          /usr/bin/git checkout '${{ github.ref }}'
+          echo "branch=${GITHUB_HEAD_REF:-${GITHUB_REF#refs/heads/}}" >> $GITHUB_OUTPUT
       - uses: actions/setup-python@v4
         with:
           python-version: '3.11.0'
-      - name: Check for file changes
-        uses: dorny/paths-filter@v2
+      - name: Check for file changes by python
         id: filter
-        with:
-          token: ${{ github.token }}
-          list-files: json
-          # The following is a single composite pattern that allows next CI steps,
-          # the pattern is in form of [not (foo or bar)] to be safe.
-          # When new files come in, the CI will NOT ignore them unless listed,
-          # so remember to extend here if they do not serve functional purposes.
-          # NOTE: careful with using ** in expression, keep !**/{old, new things}.
-          filters: |
-            cv2:
-              - "seatunnel-connectors-v2/**"
-            cv2-e2e:
-              - "seatunnel-e2e/seatunnel-connector-v2-e2e/**"
-            api:
-              - "seatunnel-api/**"
-              - "seatunnel-common/**"
-              - "seatunnel-config/**"
-              - "seatunnel-connectors/**"
-              - "seatunnel-core/**"
-              - "seatunnel-e2e/seatunnel-e2e-common/**"
-              - "seatunnel-formats/**"
-              - "seatunnel-plugin-discovery/**"
-              - "seatunnel-transforms-v2/**"
-              - "seatunnel-translation/**"
-              - "seatunnel-e2e/seatunnel-transforms-v2-e2e/**"
-              - "seatunnel-connectors/**"
-              - "pom.xml"
-              - "**/workflows/**"
-              - "**/tools/**"
-            engine:
-              - "seatunnel-engine/**"
-            engine-e2e:
-              - "seatunnel-e2e/seatunnel-engine-e2e/**"
-            deleted-poms:
-              - deleted: "**/pom.xml" 
+        run: |
+          current_branch='${{ steps.git_init.outputs.branch }}'
+          pip install GitPython
+          workspace=/home/runner/work/incubator-seatunnel/incubator-seatunnel

Review Comment:
   Please remove 'incubator'.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org