Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify option arguments #1709

Merged
merged 2 commits into from Mar 21, 2022

Conversation

shadowspawn
Copy link
Collaborator

  1. Options with a required option-argument are greedy and consume the next argument whatever the value. Options with an optional option-argument are choosy, and ignore arguments which start with a dash. We haven't written that down in the README before.

  2. Added explicit example of the ways an option and its value can be specified.

  3. Moved and slightly reworked the description of combining boolean short options, and trailing option taking a value.

@shadowspawn shadowspawn changed the base branch from master to develop March 20, 2022 11:23
@shadowspawn shadowspawn marked this pull request as ready for review March 20, 2022 11:37
Copy link
Collaborator

@abetomo abetomo left a comment

Choose a reason for hiding this comment

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

LGTM!

@shadowspawn shadowspawn merged commit 7d7a674 into tj:develop Mar 21, 2022
@shadowspawn shadowspawn deleted the feature/clarify-option-arguments branch March 21, 2022 02:06
@shadowspawn shadowspawn added the pending release Merged into a branch for a future release, but not released yet label Mar 21, 2022
@shadowspawn shadowspawn removed the pending release Merged into a branch for a future release, but not released yet label Apr 15, 2022
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.

None yet

2 participants