Skip to content

[Do not merge][Main2Main] Upgrade vllm commit to 0115#5913

Closed
wjunLu wants to merge 1 commit intovllm-project:mainfrom
wjunLu:main0115
Closed

[Do not merge][Main2Main] Upgrade vllm commit to 0115#5913
wjunLu wants to merge 1 commit intovllm-project:mainfrom
wjunLu:main0115

Conversation

@wjunLu
Copy link
Copy Markdown
Collaborator

@wjunLu wjunLu commented Jan 15, 2026

What this PR does / why we need it?

  1. ✅ Upgrade vllm commit to: 0115 (8471b27df97c3eb79f891802fc0e858f8f7ac6a0)

Test result: https://github.com/vllm-project/vllm-ascend/actions/runs/21034239336/job/60490156965?pr=5913

  1. Upgrade vllm commit to: 0116 (46f8a982b191e3a3d3a1eccaf18b184c391ac2ac)

Does this PR introduce any user-facing change?

How was this patch tested?

@github-actions github-actions bot added documentation Improvements or additions to documentation ci/build module:tests labels Jan 15, 2026
@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.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request upgrades the vLLM commit dependency. The changes include updating the commit hash in the versioning policy documentation and removing version-specific skipif decorators from tests in test_scoring.py. These changes are consistent with the goal of the pull request. The code modifications are straightforward and I did not find any issues of high or critical severity.

@wjunLu wjunLu force-pushed the main0115 branch 3 times, most recently from b84dd24 to 5481ecd Compare January 15, 2026 11:28
@wjunLu wjunLu added ready read for review ready-for-test start test by label for PR labels Jan 15, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Signed-off-by: wjunLu <wjunlu217@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@wjunLu wjunLu closed this Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build documentation Improvements or additions to documentation merge-conflicts module:tests ready read for review ready-for-test start test by label for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant