Skip to content

Commit dd1eb62

Browse files
authored
ci: resolve output variable conflict in PR test workflow (#1391)
Co-authored-by: AdityaP700 <126982848+AdityaP700@users.noreply.github.com>
1 parent 2ea86b2 commit dd1eb62

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/pr-testing-with-test-project.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,14 @@ jobs:
99
changes:
1010
runs-on: ubuntu-latest
1111
outputs:
12-
should_test: ${{ steps.filter.outputs.changes }}
12+
should_test: ${{ steps.filter.outputs.modified_files !='[]' }}
1313
steps:
1414
- uses: actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744 # v3 https://github.com/actions/checkout/releases/tag/v3
1515
- uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2 https://github.com/dorny/paths-filter/releases/tag/v3.0.2
1616
id: filter
1717
with:
1818
filters: |
19-
changes:
19+
modified_files:
2020
- 'package.json'
2121
- 'package-lock.json'
2222
- 'apps/**'
@@ -57,4 +57,4 @@ jobs:
5757
- if: steps.should_run.outputs.shouldrun == 'true'
5858
name: Run test
5959
run: NODE_IMAGE_TAG=${{ matrix.node }} docker compose up --abort-on-container-exit --remove-orphans --force-recreate
60-
working-directory: ./apps/generator/test/test-project
60+
working-directory: ./apps/generator/test/test-project

0 commit comments

Comments
 (0)