Skip to content

Conversation

@kotfu
Copy link
Member

@kotfu kotfu commented May 10, 2018

The submenu code has been extracted and added to a new project at https://github.com/python-cmd2/cmd2-submenu. The new project still needs a little work: better README, deploy to pypi, etc. However, it's far enough along that I think we should remove the code from the main cmd2 project.

@kotfu kotfu added this to the 0.9.0 milestone May 10, 2018
@kotfu kotfu requested a review from tleonhardt as a code owner May 10, 2018 00:12
@codecov
Copy link

codecov bot commented May 10, 2018

Codecov Report

Merging #397 into master will increase coverage by 0.35%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #397      +/-   ##
==========================================
+ Coverage   89.86%   90.22%   +0.35%     
==========================================
  Files           8        8              
  Lines        2645     2526     -119     
==========================================
- Hits         2377     2279      -98     
+ Misses        268      247      -21
Impacted Files Coverage Δ
cmd2/cmd2.py 91.68% <ø> (+0.66%) ⬆️

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 6a3dbec...b88b13e. Read the comment docs.

@kmvanbrunt kmvanbrunt self-requested a review May 10, 2018 01:40
Copy link
Member

@kmvanbrunt kmvanbrunt left a comment

Choose a reason for hiding this comment

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

Thanks for this change. It will make future maintenance of the core a lot easier.

pass


def _pop_readline_history(clear_history: bool=True) -> List[str]:
Copy link
Member

Choose a reason for hiding this comment

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

Looks ok to me

@tleonhardt tleonhardt merged commit 9d4d929 into master May 10, 2018
@tleonhardt tleonhardt deleted the extract_submenu branch May 10, 2018 03:29
@lv913
Copy link

lv913 commented Jul 28, 2025

Hi all, cmd2-submenu appears abandoned since 2018. It also cannot be used with cmd2 over version 2.0.
Is there any recommendation on how to implement submenus with newer cmd2 versions?

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.

5 participants