Skip tests if not making code changes

javierjulio · javierjulio · commit c957d3df7b38 · 2024-10-12T14:36:13.000-04:00
We could write this a few different ways. Fewer paths to specify if we target the paths we know we don't want to run the tests for. The focus here is on the docs and any Markdown files outside that directory (e.g. README.md or PR template in .github).
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -11,7 +11,26 @@ concurrency:
   cancel-in-progress: ${{ github.event_name == 'pull_request' }}
 
 jobs:
+  pre_test:
+    name: Pre-test confirmation
+    runs-on: ubuntu-latest
+    outputs:
+      skip_tests: ${{ steps.changed-files.outputs.only_changed }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: tj-actions/changed-files@v45
+        id: changed-files
+        with:
+          files: |
+            .github/**
+            !.github/workflows/ci.yaml
+            docs/**
+            **.md
+            .*.yml
+
   test:
+    needs: pre_test
+    if: ${{ needs.pre_test.outputs.skip_tests != 'true' }}
     name: test (${{ matrix.ruby }}, ${{ matrix.rails }})
     runs-on: ${{ matrix.os }}
     timeout-minutes: 15
