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

Fixing error in documentation #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixing error in documentation #9

wants to merge 1 commit into from

Conversation

cassava
Copy link

@cassava cassava commented May 31, 2014

I noticed that there seems to be an error in the documentation; maybe you care to merge this fix?

@ogier
Copy link
Owner

ogier commented Jun 8, 2014

That whole section of the docs is kind of crappy. The VarP example below that line is also wrong, that function doesn't take a default value and belongs inside a function not at top level. IMO that line should just be dropped entirely, VarP is pretty discoverable anyways.

@cassava
Copy link
Author

cassava commented Jun 8, 2014

True, the docs could be improved a little. Some of the text still applies (only) to the original flag package.

@ogier
Copy link
Owner

ogier commented Jun 8, 2014

Do you have any examples of this? Ideally there shouldn't be any usage of the flag package that is broken by switching to pflag, so docs that describe how to create flags the old way don't only apply to the stdlib package.

ThinkChaos pushed a commit to ThinkChaos/pflag that referenced this pull request Feb 28, 2022
Ability to mark flags as deprecated
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