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

Documentation updated for options with --no prefix #709

Merged
merged 1 commit into from Oct 26, 2017

Conversation

deepakpathania-zz
Copy link
Contributor

  • The current documentation states that multi-word options are camel-cased but doesn't clearly specify that the options with --no prefix negate the boolean value of the word that follows. This leads to a lot of confusion since the general intuition is to access it normally through program.noWord.
  • Issues Why remove "--no" from option name? #700 and No way to know if parameter is set with --no- options #630 are due to this confusion only.
  • This adds a small code sample in the main documentation clearing the confusion about the issue above.

@deepakpathania-zz
Copy link
Contributor Author

Hey @vanesyan. Can you please review this? Thanks.

@roman-vanesyan roman-vanesyan merged commit d359b75 into tj:master Oct 26, 2017
@roman-vanesyan
Copy link
Collaborator

@abetomo, you can merge commits yourself if you approve them, without asking me to approve too. 😄

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

4 participants