ci: only only skip tests if no code changes at all

.github/workflows/tests.yml

@@ -15,6 +15,7 @@ on:
  - synchronize
  - ready_for_review
  paths:
+ # Keep the list in sync with the paths defined in the `tests_skipper.yml` workflow
  - "haystack/**/*.py"
  - "haystack/templates/predefined/*"
  - "test/**/*.py"

.github/workflows/tests_skipper.yml

@@ -9,12 +9,40 @@ on:
  - synchronize
  - ready_for_review
  paths-ignore:
+ # we skip the tests unless the code changes. The problem is that GitHub will run the check anyway if any other
+ # file outside the code changed (e.g. the release notes). Hence, we need a second filter down below.
+ # keep the list in sync with the paths defined in the `tests.yml` workflow
  - "haystack/**/*.py"
+ - "haystack/templates/predefined/*"
  - "test/**/*.py"
  - "test/test_requirements.txt"
 
 jobs:
+ check_if_changed:
+ name: Check if changed
+ runs-on: ubuntu-latest
+ permissions:
+ pull-requests: read
+ outputs:
+ code_changes: ${{ steps.changes.outputs.code_changes }}
+ steps:
+ - uses: actions/checkout@v4
+ - name: Check for changed code
+ id: changes
+ uses: dorny/paths-filter@ebc4d7e9ebcb0b1eb21480bb8f43113e996ac77a
+ with:
+ # keep the list in sync with the paths defined in the `tests.yml` workflow
+ filters: |
+ code_changes:
+ - haystack/**/*.py
+ - "haystack/templates/predefined/*"
+ - test/**/*.py
+ - test/test_requirements.txt
+
  catch-all:
+ # Don't run this check if the PR contains both code and non-code changes (e.g. release notes)
+ needs: check_if_changed
+ if: needs.check_if_changed.outputs.code_changes == 'false'
  name: Catch-all check
  runs-on: ubuntu-latest
  steps: