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

Migrate command line parsing to argparse #50610

Open
wants to merge 20 commits into
base: devel
from

Conversation

1 participant
@sivel
Copy link
Member

sivel commented Jan 7, 2019

SUMMARY

Migrate command line parsing to argparse

A few things we get from this PR:

  1. subparsers, so that we don't have to hack around optparse not having subparsers
  2. better automatic help docs (--help)
  3. better handling of positional arguments
  4. optional shell completion via argcomplete
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

many

ADDITIONAL INFORMATION

@sivel

This comment has been minimized.

Copy link
Member Author

sivel commented Jan 7, 2019

bot_skip

@sivel sivel closed this Jan 8, 2019

@sivel sivel reopened this Jan 8, 2019

@sivel sivel closed this Jan 8, 2019

@sivel sivel force-pushed the sivel:argparser branch from 9266d5c to c04c29e Jan 8, 2019

@sivel sivel reopened this Jan 8, 2019

@sivel sivel closed this Jan 8, 2019

@sivel sivel reopened this Jan 8, 2019

@sivel sivel changed the title [WIP] Start of migration to argparse [WIP] Migrate command line parsing to argparse Jan 9, 2019

@sivel sivel added this to To Do - Backlog only. Anything Can be here. in Ansible 2.8 Jan 29, 2019

@sivel sivel force-pushed the sivel:argparser branch from a7e5148 to 7cca067 Feb 6, 2019

@sivel sivel force-pushed the sivel:argparser branch from e0736e1 to 7c9da74 Feb 14, 2019

@sivel sivel force-pushed the sivel:argparser branch from 7c9da74 to ce76ea5 Feb 14, 2019

@sivel sivel changed the title [WIP] Migrate command line parsing to argparse Migrate command line parsing to argparse Feb 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment