Skip to content

fix(import): clarify write-deps option in help output#10158

Merged
davidfirst merged 1 commit intomasterfrom
fix/import-write-deps-help
Jan 14, 2026
Merged

fix(import): clarify write-deps option in help output#10158
davidfirst merged 1 commit intomasterfrom
fix/import-write-deps-help

Conversation

@davidfirst
Copy link
Member

The --write-deps option was showing <workspace.jsonc|package.json> which rendered as boolean in the help output. Changed to use <target> placeholder with the valid values explained in the description.

Copilot AI review requested due to automatic review settings January 14, 2026 19:28
Copy link
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

This PR fixes the help text for the --write-deps option in the import command. The previous format <workspace.jsonc|package.json> was being incorrectly rendered as a boolean in the help output. The fix changes the placeholder to a more standard <target> format and clarifies the valid values in the description text.

Changes:

  • Updated the --write-deps option parameter placeholder from <workspace.jsonc|package.json> to <target>
  • Enhanced the description to explicitly list the valid target values ("package.json" or "workspace.jsonc") with quotes for clarity
Comments suppressed due to low confidence (1)

scopes/scope/importer/import.cmd.ts:127

  • Corrected spelling of 'dont' to 'don't'.
      'fetch dependencies (bit components) objects to the local scope, but dont add to the workspace. Useful to resolve errors about missing dependency data',

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

@davidfirst davidfirst merged commit a910ec1 into master Jan 14, 2026
18 checks passed
@davidfirst davidfirst deleted the fix/import-write-deps-help branch January 14, 2026 20:21
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.

3 participants