Skip to content

Commit 9ff51cb

Browse files
authored
Merge file change tests for Python source and requirements files (#1206)
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 9ff51cb

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)