Skip to content

Fix extra deps in docker - #409

Merged
esmeetu merged 1 commit into
mainfrom
fix-extra
Apr 28, 2026
Merged

Fix extra deps in docker#409
esmeetu merged 1 commit into
mainfrom
fix-extra

Conversation

@esmeetu

@esmeetu esmeetu commented Apr 28, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: yasong.wang <yasong.wang@inferact.ai>
@vercel

vercel Bot commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
vllm-recipes Ready Ready Preview, Comment Apr 28, 2026 3:32am

Request Review

@esmeetu
esmeetu merged commit be10da8 into main Apr 28, 2026
3 of 4 checks passed
@esmeetu
esmeetu deleted the fix-extra branch April 28, 2026 03:33

@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 CommandBuilder component to conditionally hide the DependenciesBlock when the installation mode is set to "docker". This change ensures that host-level installation instructions, such as pip or bash scripts, are not displayed when the user is utilizing a container image. I have no feedback to provide.

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.

1 participant