Skip to content

Commit 103afb4

Browse files
committed
build: more files need to be checked for changes
1 parent f9e2afd commit 103afb4

File tree

3 files changed

+17
-18
lines changed

3 files changed

+17
-18
lines changed

.github/workflows/coverage.yml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
permissions:
3535
pull-requests: read
3636
outputs:
37-
python: ${{ steps.filter.outputs.python }}
37+
run_coverage: ${{ steps.filter.outputs.run_coverage }}
3838
workflow: ${{ steps.filter.outputs.workflow }}
3939
steps:
4040
- name: "Check out the repo"
@@ -47,10 +47,12 @@ jobs:
4747
id: filter
4848
with:
4949
filters: |
50-
python:
50+
run_coverage:
5151
- "**.py"
52-
workflow:
5352
- ".github/workflows/coverage.yml"
53+
- "tox.ini"
54+
- "requirements/*.pip"
55+
- "tests/gold/**"
5456
5557
coverage:
5658
name: "${{ matrix.python-version }} on ${{ matrix.os }}"
@@ -59,11 +61,7 @@ jobs:
5961

6062
# Only run coverage if Python files or this workflow changed.
6163
needs: changed
62-
if: |
63-
${{
64-
needs.changed.outputs.python == 'true'
65-
|| needs.changed.outputs.workflow == 'true'
66-
}}
64+
if: ${{ needs.changed.outputs.run_coverage == 'true' }}
6765

6866
env:
6967
MATRIX_ID: "${{ matrix.python-version }}.${{ matrix.os }}"

.github/workflows/quality.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,13 @@ jobs:
5151
- "**.py"
5252
docs:
5353
- "doc/**"
54+
- "coverage/**.py"
5455
actions:
5556
- ".github/workflows/**"
5657
workflow:
5758
- ".github/workflows/quality.yml"
59+
- "tox.ini"
60+
- "requirements/*.pip"
5861
5962
lint:
6063
name: "Pylint etc"

.github/workflows/testsuite.yml

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,7 @@ jobs:
3434
permissions:
3535
pull-requests: read
3636
outputs:
37-
python: ${{ steps.filter.outputs.python }}
38-
workflow: ${{ steps.filter.outputs.workflow }}
37+
run_tests: ${{ steps.filter.outputs.run_tests }}
3938
steps:
4039
- name: "Check out the repo"
4140
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
@@ -47,26 +46,25 @@ jobs:
4746
id: filter
4847
with:
4948
filters: |
50-
python:
49+
run_tests:
5150
- "**.py"
52-
workflow:
5351
- ".github/workflows/testsuite.yml"
52+
- "tox.ini"
53+
- "requirements/*.pip"
54+
- "tests/gold/**"
5455
5556
tests:
5657
name: "${{ matrix.python-version }} on ${{ matrix.os }}"
5758
runs-on: "${{ matrix.os }}-${{ matrix.os-version || 'latest' }}"
5859
timeout-minutes: 30
5960

6061
# Don't run tests if the branch name includes "-notests".
61-
# Only run tests if Python files or this workflow changed.
62+
# Only run tests if files that affect tests have changed.
6263
needs: changed
6364
if: |
6465
${{
65-
!contains(github.ref, '-notests')
66-
&& (
67-
needs.changed.outputs.python == 'true'
68-
|| needs.changed.outputs.workflow == 'true'
69-
)
66+
needs.changed.outputs.run_tests == 'true'
67+
&& !contains(github.ref, '-notests')
7068
}}
7169
7270
strategy:

0 commit comments

Comments
 (0)