Skip to content

Conversation

@kmvanbrunt
Copy link
Member

Fixed issue where subcommand usage text could contain a subcommand alias instead of the actual name

@codecov
Copy link

codecov bot commented Apr 23, 2020

Codecov Report

Merging #928 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #928   +/-   ##
=======================================
  Coverage   97.78%   97.79%           
=======================================
  Files          16       16           
  Lines        4072     4077    +5     
=======================================
+ Hits         3982     3987    +5     
  Misses         90       90           
Impacted Files Coverage Δ
cmd2/decorators.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0bc9217...6545429. Read the comment docs.

tleonhardt
tleonhardt previously approved these changes Apr 23, 2020
Copy link
Member

@tleonhardt tleonhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look OK to me

…he correct prog value

This makes dynamically adding subcommands after the CLI starts easier.
@kmvanbrunt kmvanbrunt merged commit ecf154f into master Apr 23, 2020
@kmvanbrunt kmvanbrunt deleted the subcmd_aliases branch April 23, 2020 19:40
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.

3 participants