Skip to content

docs: Improve docs for vpx command.#1303

Open
connorshea wants to merge 2 commits intovoidzero-dev:mainfrom
connorshea:improve-vpx-docs
Open

docs: Improve docs for vpx command.#1303
connorshea wants to merge 2 commits intovoidzero-dev:mainfrom
connorshea:improve-vpx-docs

Conversation

@connorshea
Copy link
Copy Markdown
Contributor

Just making it clear that it will install the package if it's not found, rather than just saying it "can" download it when needed.

Just making it clear that it will install the package if it's not found, rather than just saying it "can" download it when needed.
Copilot AI review requested due to automatic review settings April 5, 2026 01:49
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 5, 2026

Deploy Preview for viteplus-preview ready!

Name Link
🔨 Latest commit 5c20419
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69d1c80941f78e00084874d9
😎 Deploy Preview https://deploy-preview-1303--viteplus-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Clarifies vpx documentation to better describe its local-first execution and remote fallback behavior when the requested binary isn’t available locally.

Changes:

  • Updates the overview bullet to say vpx “will download” when a binary isn’t found locally.
  • Refines the --package/-p option description to mention “additional packages”.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

### Options

- `-p, --package <name>` installs one or more packages before running the command
- `-p, --package <name>` installs one or more additional packages before running the command
Copy link

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

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

The updated --package/-p description says “additional packages”, but vpx supports using --package to specify the package that provides the command when the binary name differs from the package name (e.g. vpx --package @pnpm/meta-updater meta-updater --help). Calling these “additional” can be misleading; consider rewording to “installs one or more packages before running the command” (or similar) so it covers both primary and extra packages.

Suggested change
- `-p, --package <name>` installs one or more additional packages before running the command
- `-p, --package <name>` installs one or more packages before running the command

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm actually not sure why the previous docs were written this way about -p/--package if this is true? Were these docs AI-generated or is Copilot wrong 😅

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Connor Shea <connor.james.shea@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.

2 participants