Skip to content

Conversation

@kmvanbrunt
Copy link
Member

@kmvanbrunt kmvanbrunt commented Jan 6, 2020

Updated example to work with categorizing dynamic commands
Fixes #835

@kmvanbrunt kmvanbrunt requested a review from tleonhardt as a code owner January 6, 2020 17:10
@codecov
Copy link

codecov bot commented Jan 6, 2020

Codecov Report

Merging #840 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #840   +/-   ##
=======================================
  Coverage   97.35%   97.35%           
=======================================
  Files          14       14           
  Lines        3557     3557           
=======================================
  Hits         3463     3463           
  Misses         94       94

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 6b6809a...5dac940. Read the comment docs.

from cmd2.constants import COMMAND_FUNC_PREFIX, HELP_FUNC_PREFIX

COMMAND_LIST = ['foo', 'bar']
CATEGORY = 'Dynamic Commands'
Copy link
Member

Choose a reason for hiding this comment

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

This is a very nice improvement to the example. Thanks!

@kmvanbrunt kmvanbrunt merged commit 162603e into master Jan 6, 2020
@tleonhardt tleonhardt deleted the categorize_dynamic branch January 6, 2020 18:33
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.

Categorizing Dynamic Commands

3 participants