- Notifications
You must be signed in to change notification settings - Fork384
Use argparse cmd line parser instead of optparse#2
Merged
fabianp merged 2 commits intopythonprofilers:masterfromMay 13, 2012
Merged
Use argparse cmd line parser instead of optparse#2fabianp merged 2 commits intopythonprofilers:masterfrom
fabianp merged 2 commits intopythonprofilers:masterfrom
Conversation
According to the docs, optparse is deprecated(http://docs.python.org/py3k/library/optparse.html), so that leaves uswith argparse.Rundown of what was done:+ removed arguments: * -o since output can just be redirected * -l and -v they're not necessary+ made the filename a positional argument+ made the necessary changes to accomodate for the above
Collaborator
fabianp commentedMay 13, 2012
great, it's in. |
fabianp pushed a commit that referenced this pull requestMay 13, 2012
Use argparse cmd line parser instead of optparse
fabianp added a commit that referenced this pull requestOct 29, 2014
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Labels
None yet
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
optparseis deprecated, this changes it toargparse. Also, there are some other small changes that have been made, such as getting rid of-o,-l, and-v; and making the filename a positional argument.