Skip to content

[Bugfix] Apply ruff-format to hyperclovax.py - #41620

Merged
vllm-bot merged 1 commit into
vllm-project:mainfrom
stecasta:fix/hyperclovax-ruff-format
May 4, 2026
Merged

vllm-bot merged 1 commit into
vllm-project:mainfrom
stecasta:fix/hyperclovax-ruff-format

Conversation

@stecasta

@stecasta stecasta commented May 4, 2026

Copy link
Copy Markdown
Contributor

ruff-format flags two lines in vllm/transformers_utils/configs/hyperclovax.py (vendored in #38447). Collapses the AutoConfig.for_model(...) call to one line.

Verified: pre-commit run ruff-format --files vllm/transformers_utils/configs/hyperclovax.py.

Signed-off-by: Stefano Castagnetta scastagnetta@nvidia.com

Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>

@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 bug Something isn't working label May 4, 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 simplifies the formatting of the HyperCLOVA X configuration by consolidating a multi-line call to AutoConfig.for_model into a single line. I have no feedback to provide.

@stecasta

stecasta commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

@DarkLight1337

@vllm-bot
vllm-bot merged commit 6f53753 into vllm-project:main May 4, 2026
9 of 10 checks passed
Copilot AI pushed a commit to hongbolv/vllm that referenced this pull request May 7, 2026
Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>
Co-authored-by: hongbolv <33214277+hongbolv@users.noreply.github.com>
weifang231 pushed a commit to weifang231/eb-vllm that referenced this pull request May 13, 2026
Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>
mfylcek pushed a commit to mfylcek/vllm that referenced this pull request May 19, 2026
Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants