Skip to content

Commit 783c3f8

Browse files
authored
Make CI run on any changes to tools, check scripts, and all workflow files (#1122)
If a script in `./check/` or `./dev_tools/` is changed, it may affect the results of running CI tests. To account for that, this PR expands the file change filter to detect any changes in those directories. In addition, this PR widens the files tested in `.github/workflows` to include any file changes. The result is that any change in the dev_tools, scripts, or the CI workflows will trigger all the tests, not only the file-type-specific tests.
1 parent 3d8e3ea commit 783c3f8

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

.github/workflows/ci.yaml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -100,8 +100,9 @@ jobs:
100100
gha: ${{steps.filter.outputs.gha}}
101101
gha_files: ${{steps.filter.outputs.gha_files}}
102102

103-
# The following all test both the relevant file condition & the CI config
104-
# because a change in the CI workflows can affect the CI check results.
103+
# The rest all test both the relevant files and ALSO tools & scripts b/c
104+
# a change in the CI workflows or scripts can affect the check results.
105+
105106
python: ${{steps.filter.outputs.python || steps.filter.outputs.ci}}
106107
python_files: ${{steps.filter.outputs.python_files}}
107108

@@ -159,9 +160,10 @@ jobs:
159160
# The outputs will be variables named "foo_files" for a filter "foo".
160161
filters: |
161162
ci:
162-
- './.github/workflows/ci.yml'
163-
- './.github/workflows/codeql.yaml'
164-
- './.github/workflows/osv-scanner.yaml'
163+
- './.github/workflows/**'
164+
- './.github/problem-matchers/**'
165+
- './check/**'
166+
- './dev_tools/**'
165167
cff:
166168
- added|modified:
167169
- '**/CITATION.cff'

0 commit comments

Comments
 (0)