Skip to content

fix: fix vllm test skip - #394

Merged
akihikokuroda merged 1 commit into
generative-computing:mainfrom
akihikokuroda:skipvllm
Feb 5, 2026
Merged

fix: fix vllm test skip#394
akihikokuroda merged 1 commit into
generative-computing:mainfrom
akihikokuroda:skipvllm

Conversation

@akihikokuroda

@akihikokuroda akihikokuroda commented Jan 31, 2026

Copy link
Copy Markdown
Contributor

Misc PR

Type of PR

  • Bug Fix
  • New Feature
  • Documentation
  • Other

Description

Testing

  • Tests added to the respective file if code was changed
  • New code has 100% coverage if code as added
  • Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

@github-actions

Copy link
Copy Markdown
Contributor

The PR description has been updated. Please fill out the template for your PR to be reviewed.

@mergify

mergify Bot commented Jan 31, 2026

Copy link
Copy Markdown

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|release)(?:\(.+\))?:

@nrfulton nrfulton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@planetf1 planetf1 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.

LGTM. good to get it merged as we iterate on improving test usability in differing configurations

Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
akihikokuroda added this pull request to the merge queue Feb 5, 2026
Merged via the queue into generative-computing:main with commit 123db62 Feb 5, 2026
4 checks passed
akihikokuroda added a commit to akihikokuroda/mellea that referenced this pull request May 27, 2026
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vllm tests failed without vllm install

3 participants