Skip to content

Commit dbe3983

Browse files
committed
Merge file change tests for Python source and requirements files
The requirements file test results were never used, and what's worse, were ignored as a sign of pytest having to be re-run. Putting the requirements files in with the Python file change tests solves both problems.
1 parent 50eca79 commit dbe3983

1 file changed

Lines changed: 5 additions & 7 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,6 @@ jobs:
118118
docker: ${{steps.filter.outputs.docker || steps.filter.outputs.ci}}
119119
docker_files: ${{steps.filter.outputs.docker_files}}
120120

121-
requirements: ${{steps.filter.outputs.requirements || steps.filter.outputs.ci}}
122-
requirements_files: ${{steps.filter.outputs.requirements_files}}
123-
124121
shell: ${{steps.filter.outputs.shell || steps.filter.outputs.ci}}
125122
shell_files: ${{steps.filter.outputs.shell_files}}
126123
steps:
@@ -169,6 +166,11 @@ jobs:
169166
- '**/CITATION.cff'
170167
python:
171168
- '**/*.py'
169+
- 'dev_tools/requirements/deps/*.txt'
170+
- 'dev_tools/requirements/env/*.txt'
171+
- 'dev_tools/requirements/max_compat/*.txt'
172+
- 'docs/**/*-requirements.txt'
173+
- 'docs/**/requirements.txt'
172174
gha:
173175
- added|modified:
174176
- './.github/workflows/*.yaml'
@@ -183,10 +185,6 @@ jobs:
183185
docker:
184186
- '**/dockerfile'
185187
- '**/Dockerfile'
186-
requirements:
187-
- 'dev_tools/requirements/deps/*.txt'
188-
- 'docs/**/*-requirements.txt'
189-
- 'docs/**/requirements.txt'
190188
shell:
191189
- '**/*.sh'
192190
- 'check/*'

0 commit comments

Comments
 (0)