chore(ci/pr-check_redirects): remove path filter to always run the re…

…quired test (#16903)
mdn · Nov 17, 2023 · 53dbdb2 · 53dbdb2
1 parent f5ad8f0
commit 53dbdb2
Showing 1 changed file with 17 additions and 4 deletions.
diff --git a/.github/workflows/pr-check_redirects.yml b/.github/workflows/pr-check_redirects.yml
@@ -4,10 +4,6 @@ on:
   pull_request:
     branches:
       - main
-    paths:
-      - .nvmrc
-      - files/**
-      - .github/workflows/pr-check_redirects.yml
 
 jobs:
   check_redirects:
@@ -18,19 +14,35 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
+      # This is a "required" workflow so path filtering can not be used:
+      # https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks
+      # We have to rely on a custom filtering mechanism to run the checks only if required files are modified.
+      - uses: dorny/paths-filter@v2
+        name: See if any file needs checking
+        id: filter
+        with:
+          filters: |
+            required_files:
+              - ".nvmrc"
+              - "files/**"
+              - ".github/workflows/pr-check_redirects.yml"
+
       - uses: actions/checkout@v4
+        if: steps.filter.outputs.required_files == 'true'
         with:
           repository: mdn/content
           path: mdn/content
 
       - name: Setup Node.js environment
+        if: steps.filter.outputs.required_files == 'true'
         uses: actions/setup-node@v4
         with:
           node-version-file: ".nvmrc"
           cache: "yarn"
           cache-dependency-path: mdn/content/yarn.lock
 
       - name: Install all yarn packages
+        if: steps.filter.outputs.required_files == 'true'
         working-directory: ${{ github.workspace }}/mdn/content
         run: |
           yarn --frozen-lockfile
@@ -39,6 +51,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Check redirects file(s)
+        if: steps.filter.outputs.required_files == 'true'
         env:
           CONTENT_ROOT: ${{ github.workspace }}/mdn/content/files
           CONTENT_TRANSLATED_ROOT: ${{ github.workspace }}/files