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

Use argparse cmd line parser instead of optparse #2

Merged
merged 2 commits into from
May 13, 2012

Conversation

octavo
Copy link
Contributor

@octavo octavo commented May 12, 2012

optparse is deprecated, this changes it to argparse. 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.

Victor added 2 commits May 12, 2012 17:10
According to the docs, optparse is deprecated
(http://docs.python.org/py3k/library/optparse.html), so that leaves us
with 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
@fabianp
Copy link
Collaborator

fabianp commented May 13, 2012

great, it's in.

fabianp pushed a commit that referenced this pull request May 13, 2012
Use argparse cmd line parser instead of optparse
@fabianp fabianp merged commit f2a308a into pythonprofilers:master May 13, 2012
fabianp added a commit that referenced this pull request Oct 29, 2014
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