search

vercel / arg

Simple argument parsing
https://npmjs.com/arg
MIT License
1.23k stars 54 forks source link

Add literalDashes option #35

Closed Qix- closed 5 years ago

Qix- commented 5 years ago

Closes #30. Supercedes #33.

This adds the literalDashes option as explained in https://github.com/zeit/arg/issues/30#issuecomment-450615316.

I also filled in a few pieces of missing documentation.

Qix- commented 5 years ago

cc @blakeembrey - what do you think?

rauchg commented 5 years ago

This also adds stopAtPositional ? Bit confusing

Qix- commented 5 years ago

@rauchg just the documentation. It wasn't included in the PR that added the functionality. I would have separated it out into its own commit but it'd be useless since only Squash & Merge is enabled for the repo.

Qix- commented 5 years ago

Even though #30 was closed, this is still a nice option to have. IMO one of the few options that should exist.

blakeembrey commented 5 years ago

@Qix- Do you know when or why someone would use this options? Just curious since it seems to go against the POSIX guidelines you were mentioning (e.g. https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap12.html).

This PR functionality isn't something I need or would use, but don't let it stop anything 👍

Qix- commented 5 years ago

... You're right. I can't think of one, for some reason I had it in my head that -- was used in a few ways but right now I can't. Seeing the POSIX guidelines kind of confirms that.

I'll break off the missing docs into their own PR. Thanks @blakeembrey :)