Skip to content

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