Skip to content

[CI] Add tests/parser to CI coverage#41877

Merged
chaunceyjiang merged 2 commits into
vllm-project:mainfrom
sfeng33:parser-ci
May 11, 2026
Merged

[CI] Add tests/parser to CI coverage#41877
chaunceyjiang merged 2 commits into
vllm-project:mainfrom
sfeng33:parser-ci

Conversation

@sfeng33

@sfeng33 sfeng33 commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Add tests/parser coverage in CI in the CPU only test group.

Signed-off-by: sfeng33 <4florafeng@gmail.com>
@sfeng33 sfeng33 requested a review from chaunceyjiang May 6, 2026 23:02
@sfeng33 sfeng33 marked this pull request as ready for review May 6, 2026 23:03

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@mergify mergify Bot added the ci/build label May 6, 2026

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

Copy link
Copy Markdown
Contributor

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 updates the Buildkite CI configurations in .buildkite/test-amd.yaml and .buildkite/test_areas/misc.yaml to include the tests/parser directory and execute its corresponding pytest command. I have no feedback to provide as there were no review comments.

@chaunceyjiang chaunceyjiang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@chaunceyjiang chaunceyjiang added the ready ONLY add when PR is ready to merge/full CI is needed label May 8, 2026
@chaunceyjiang chaunceyjiang enabled auto-merge (squash) May 8, 2026 03:29
@sfeng33 sfeng33 requested a review from Harry-Chen as a code owner May 11, 2026 16:37
@chaunceyjiang chaunceyjiang merged commit 639cbfd into vllm-project:main May 11, 2026
24 checks passed
@sfeng33 sfeng33 deleted the parser-ci branch May 11, 2026 19:22
weifang231 pushed a commit to weifang231/eb-vllm that referenced this pull request May 13, 2026
Signed-off-by: sfeng33 <4florafeng@gmail.com>
mfylcek pushed a commit to mfylcek/vllm that referenced this pull request May 19, 2026
Signed-off-by: sfeng33 <4florafeng@gmail.com>
jhu960213 pushed a commit to jhu960213/vllm that referenced this pull request May 20, 2026
Signed-off-by: sfeng33 <4florafeng@gmail.com>
h1t35h pushed a commit to h1t35h/vllm that referenced this pull request May 21, 2026
Signed-off-by: sfeng33 <4florafeng@gmail.com>
mvanhorn pushed a commit to mvanhorn/vllm that referenced this pull request Jun 4, 2026
Signed-off-by: sfeng33 <4florafeng@gmail.com>
Signed-off-by: Matt Van Horn <455140+mvanhorn@users.noreply.github.com>
knight0528 pushed a commit to knight0528/vllm that referenced this pull request Jun 8, 2026
Signed-off-by: sfeng33 <4florafeng@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants