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.

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

Fix release helper subcommands #6451

Merged

Conversation

Jackenmen
Copy link
Member

Description of the changes

It turns out that you need to check that ctx.invoked_subcommand is not None in the parent command.

Have the changes in this PR been tested?

Yes

@Jackenmen Jackenmen added Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. Changelog Entry: Skipped Changelog entry for this PR is unnecessary. labels Sep 15, 2024
@github-actions github-actions bot added Category: Meta This is related to the repository maintenance. labels Sep 15, 2024
@Jackenmen Jackenmen merged commit d3887b5 into Cog-Creators:V3/develop Sep 15, 2024
19 checks passed
@Jackenmen Jackenmen deleted the fix_release_helper_subcommands branch September 15, 2024 01:02
BenCos17 added a commit to JARVIS-discordbot/Red-DiscordBot-jarvis that referenced this pull request Sep 24, 2024
* Fix release helper subcommands (Cog-Creators#6451)

* Fix for UIBs regarding permission fetching

---------

Co-authored-by: Jakub Kuczys <me@jacken.men>
Co-authored-by: Kowlin <git@wyvern.blue>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Meta This is related to the repository maintenance. Changelog Entry: Skipped Changelog entry for this PR is unnecessary. Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant