Skip to content

Commit 3a38449

Browse files
authored
[CI] add lint block before running e2e (#2447)
### What this PR does / why we need it? add lint block before running e2e. follow up #2445 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? N/A Signed-off-by: MengqingCao <[email protected]>
1 parent 1327f9b commit 3a38449

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/vllm_ascend_test.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,9 +130,9 @@ jobs:
130130
verbose: true
131131

132132
e2e:
133-
needs: [changes]
133+
needs: [lint, changes]
134134
# only trigger e2e test after lint passed and the change is e2e related with pull request.
135-
if: ${{ github.event_name == 'pull_request' && needs.changes.outputs.e2e_tracker == 'true' }}
135+
if: ${{ github.event_name == 'pull_request' && needs.lint.result == 'success' && needs.changes.outputs.e2e_tracker == 'true' }}
136136
strategy:
137137
max-parallel: 2
138138
matrix:

0 commit comments

Comments
 (0)