Skip to content

docs: fix default argument placeholder syntax - #254

Merged
bcomnes merged 1 commit into
bcomnes:masterfrom
noritaka1166:fix-default-placeholder-syntax
Oct 20, 2026
Merged

bcomnes merged 1 commit into
bcomnes:masterfrom
noritaka1166:fix-default-placeholder-syntax

Conversation

@noritaka1166

Copy link
Copy Markdown
Contributor

Summary

  • Correct the default argument placeholder from {1-=foo} to {1:-foo} in the npm-run-all, run-s, and run-p documentation.
  • Align the examples with the existing implementation and tests.

@bcomnes

bcomnes commented Sep 26, 2026

Copy link
Copy Markdown
Owner

Need to double check this isn't package manager variance.

@bcomnes
bcomnes requested a review from voxpelli October 20, 2026 23:50

@bcomnes bcomnes left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks!

@bcomnes
bcomnes merged commit ff7d461 into bcomnes:master Oct 20, 2026
17 checks passed
@noritaka1166
noritaka1166 deleted the fix-default-placeholder-syntax branch October 20, 2026 23:51
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