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.

Sign up for GitHub

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

Jump to bottom

Restore showing subcommand list #288

Merged
merged 2 commits into from Jul 19, 2023
Merged

Restore showing subcommand list #288

merged 2 commits into from Jul 19, 2023

Conversation

dnr
Copy link
Member

@dnr dnr commented Jul 19, 2023 •
edited

What was changed

Fix help output to show list of subcommands again. This was broken by #280 by something in the cli update.

Why?

Self-documentation

Checklist

  1. Closes

  2. How was this tested:
    manually + unit tests (skipped for now)

  3. Any docs updates needed?

@dnr dnr requested a review from feedmeapples July 19, 2023 20:19
@feedmeapples
Copy link
Contributor

feedmeapples commented Jul 19, 2023 •
edited

thx, added tests

@dnr
Copy link
Member Author

dnr commented Jul 19, 2023

thanks for the tests, that looks good

@dnr dnr merged commit f78339c into main Jul 19, 2023
15 checks passed
@dnr dnr deleted the david/showcommands branch July 19, 2023 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feedmeapples feedmeapples feedmeapples approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dnr @feedmeapples