Skip to content

fix: put sub-command text as part of positional argument help text#1719

Merged
tleonhardt merged 1 commit into
mainfrom
fix/annotated-subparser-positional
Jul 16, 2026
Merged

fix: put sub-command text as part of positional argument help text#1719
tleonhardt merged 1 commit into
mainfrom
fix/annotated-subparser-positional

Conversation

@KelvinChung2000

@KelvinChung2000 KelvinChung2000 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

close #1715

Before:

Usage: base [-h] SUBCOMMAND ...

Manage the widgets.

Options:
  -h, --help   show this help message and exit

  SUBCOMMAND
    add        add a widget
    list (ls)  list widgets

After:

Usage: base [-h] SUBCOMMAND ...

Manage the widgets.

Positional Arguments:
  SUBCOMMAND
    add        add a widget
    list (ls)  list widgets

Options:
  -h, --help   show this help message and exit

@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.57%. Comparing base (cb4e6c5) to head (cde81ec).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1719      +/-   ##
==========================================
- Coverage   99.61%   99.57%   -0.04%     
==========================================
  Files          23       23              
  Lines        5906     5910       +4     
==========================================
+ Hits         5883     5885       +2     
- Misses         23       25       +2     
Flag Coverage Δ
unittests 99.57% <100.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@tleonhardt
tleonhardt merged commit 8c036f7 into main Jul 16, 2026
28 of 29 checks passed
@tleonhardt
tleonhardt deleted the fix/annotated-subparser-positional branch July 16, 2026 22:12
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.

[with_annotated] Subparser should be listed as positional arguments by default

2 participants