nits: a couple small tweaks while merging upstream

pkgjs / parseargs

Polyfill of `util.parseArgs()`

Apache License 2.0

121 stars 9 forks source link

nits: a couple small tweaks while merging upstream #117

Closed bcoe closed 2 years ago

bcoe commented 2 years ago

A couple small tweaks made while synchronizing with Node.js repository.

small tweak to wording for greedy args (thought it read clearer).
received warning from linter for capital Object in JSDoc.

shadowspawn commented 2 years ago

An aside: with a lot of recent suggestions on the node PR, would you like to make those changes upstream and later copy-down to parseArgs? Or make changes here and copy-up? (Whatever is easiest.)

bcoe commented 2 years ago

An aside: with a lot of recent suggestions on the node PR, would you like to make those changes upstream and later copy-down to parseArgs? Or make changes here and copy-up? (Whatever is easiest.)

I'll just make a new PR with the upstream changes.