Skip to content

[CI] Align coverage workflow with PR-test vLLM version - #16220

Merged
wangxiyuan merged 1 commit into
vllm-project:mainfrom
shiqiangA:coverage_fix
Sep 10, 2026
Merged

wangxiyuan merged 1 commit into
vllm-project:mainfrom
shiqiangA:coverage_fix

Conversation

@shiqiangA

@shiqiangA shiqiangA commented Sep 10, 2026 •

Copy link
Copy Markdown
Collaborator

Use the verified vLLM main commit from pr_test.yaml, and allow the coverage workflow to run on pull requests.

What this PR does / why we need it?

The version of the scheduled task vllm is changed to be the same as that of the PR.

Does this PR introduce any user-facing change?

no

How was this patch tested?

The version of the VLLM used by the coverage rate scheduled task is the same as that used by the PR.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:‌‌

  • A PR should do only one thing, smaller PRs enable faster reviews.
  • Every PR should include unit tests and end-to-end tests ‌to ensure it works and is not broken by other future PRs.
  • Write the commit message by fulfilling the PR description to help reviewer and future developers understand.

If CI fails, you can run linting and testing checks locally according Contributing and Testing.

Use the verified vLLM main commit from pr_test.yaml instead of the release tag.

Signed-off-by: shiqiangA <shiqiang15@huawei.com>
@wangxiyuan
wangxiyuan merged commit 9897c35 into vllm-project:main Sep 10, 2026
13 checks passed
sunny-rain-63 pushed a commit to sunny-rain-63/vllm-ascend that referenced this pull request Sep 12, 2026
…16220)

Use the verified vLLM main commit from pr_test.yaml, and allow the
coverage workflow to run on pull requests.

### What this PR does / why we need it?
The version of the scheduled task vllm is changed to be the same as that
of the PR.
### Does this PR introduce _any_ user-facing change?
no
### How was this patch tested?
The version of the VLLM used by the coverage rate scheduled task is the
same as that used by the PR.
- vLLM main:
vllm-project/vllm@b2f6858
johnnysluckydays pushed a commit to johnnysluckydays/vllm-ascend that referenced this pull request Sep 14, 2026
…16220)

Use the verified vLLM main commit from pr_test.yaml, and allow the
coverage workflow to run on pull requests.

### What this PR does / why we need it?
The version of the scheduled task vllm is changed to be the same as that
of the PR.
### Does this PR introduce _any_ user-facing change?
no
### How was this patch tested?
The version of the VLLM used by the coverage rate scheduled task is the
same as that used by the PR.
- vLLM main:
vllm-project/vllm@b2f6858
Signed-off-by: tianming2009 <13246728590@163.com>
like-0517 pushed a commit to like-0517/vllm-ascend that referenced this pull request Sep 15, 2026
…16220)

Use the verified vLLM main commit from pr_test.yaml, and allow the
coverage workflow to run on pull requests.

### What this PR does / why we need it?
The version of the scheduled task vllm is changed to be the same as that
of the PR.
### Does this PR introduce _any_ user-facing change?
no
### How was this patch tested?
The version of the VLLM used by the coverage rate scheduled task is the
same as that used by the PR.
- vLLM main:
vllm-project/vllm@b2f6858

Signed-off-by: like-0517 <ithwlike@126.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants