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

feat: Write cappa backend/parser. #3

Merged
merged 1 commit into from
Oct 9, 2023
Merged

feat: Write cappa backend/parser. #3

merged 1 commit into from
Oct 9, 2023

Conversation

DanCardin
Copy link
Owner

@DanCardin DanCardin commented Oct 9, 2023

  • Adds native cappa parser option vs argparse
  • Renames render option to "backend"
  • Collapse separate Subcommands abstraction into Subcommand
  • Renames Subcmd to Subcommands
  • Adds support for autocompletion when using new parser
  • Removes "rich" extra
    • Adds rich as a required extra dependency
      • It is core to custom/colored help formatting for new parser, and the now more-unified stdout/stderr output.
    • rich-argparse needs to now be installed separately, if used
  • Adds a new Output implicit dependency option
  • Adds new theme option (currently only applied to custom parser)
  • Adds group option to Arg to control how arguments are grouped together in help text
  • Adds hidden option to Arg, to hide arguments from help text.
  • Ensures help= applies to all levels of subcommands
  • Fixes Confusing/unclear parsing output #2

@coveralls
Copy link

Pull Request Test Coverage Report for Build 6462821733

  • 685 of 685 (100.0%) changed or added relevant lines in 17 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.05%) to 99.951%

Files with Coverage Reduction New Missed Lines %
src/cappa/argparse.py 1 99.5%
Totals Coverage Status
Change from base Build 6355604095: -0.05%
Covered Lines: 1452
Relevant Lines: 1453

💛 - Coveralls

@DanCardin DanCardin merged commit 38716fb into main Oct 9, 2023
6 checks passed
@DanCardin DanCardin deleted the dc/parser branch October 9, 2023 23:47
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.

Confusing/unclear parsing output
2 participants