Skip to content

Commit bf25498

Browse files
authored
[CI] Fix changes CI to recover codecov (#1799)
Add `checkout` action before `dorny/paths-filter` to make it works with `push` case. This is a known issue that `dorny/paths-filter` works without `checkout` in `pull_request` case but failed in `push` case. More detail is here: dorny/paths-filter#60 (comment) The push CI works after this PR. The test result is here: https://github.com/wangxiyuan/vllm-ascend/actions/runs/16285606468/job/45983607539 - vLLM version: v0.9.2 - vLLM main: vllm-project/vllm@d4d3094 Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
1 parent 787010a commit bf25498

File tree

1 file changed

+19
-18
lines changed

1 file changed

+19
-18
lines changed

.github/workflows/vllm_ascend_test.yaml

Lines changed: 19 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -49,24 +49,25 @@ jobs:
4949
e2e_tracker: ${{ steps.filter.outputs.e2e_tracker }}
5050
ut_tracker: ${{ steps.filter.outputs.ut_tracker }}
5151
steps:
52-
- uses: dorny/paths-filter@v3
53-
id: filter
54-
with:
55-
filters: |
56-
e2e_tracker:
57-
- '.github/workflows/vllm_ascend_test.yaml'
58-
- 'vllm_ascend/**'
59-
- 'csrc/**'
60-
- 'cmake/**'
61-
- 'tests/e2e/**'
62-
- 'CMakeLists.txt'
63-
- 'setup.py'
64-
- 'requirements.txt'
65-
- 'requirements-dev.txt'
66-
- 'requirements-lint.txt'
67-
- 'packages.txt'
68-
ut_tracker:
69-
- 'tests/ut/**'
52+
- uses: actions/checkout@v4
53+
- uses: dorny/paths-filter@v3
54+
id: filter
55+
with:
56+
filters: |
57+
e2e_tracker:
58+
- '.github/workflows/vllm_ascend_test.yaml'
59+
- 'vllm_ascend/**'
60+
- 'csrc/**'
61+
- 'cmake/**'
62+
- 'tests/e2e/**'
63+
- 'CMakeLists.txt'
64+
- 'setup.py'
65+
- 'requirements.txt'
66+
- 'requirements-dev.txt'
67+
- 'requirements-lint.txt'
68+
- 'packages.txt'
69+
ut_tracker:
70+
- 'tests/ut/**'
7071
ut:
7172
needs: [lint, changes]
7273
name: unit test

0 commit comments

Comments
 (0)