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

Changed CLI to use an argparse-based parser #115

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

reklawnos
Copy link
Member

From previous pull request that got reverted:

The "commands" file was pretty messy, so I put together an argparse parser for command line arguments. It'll handle improper number of arguments/optional arguments/help text for us. There is only one breaking change I noticed which is that listversions now takes a -d or --documents flag for showing document versions, so instead of ./freejournal_cli.py listversions documents, it's ./freejournal_cli.py listversions -d or ./freejournal_cli.py listversions --documents.

@pdaian, can you take a look at the help messages for the commands and make sure they look reasonable? Additionally, if naming is off for anything just let me know.

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.

1 participant