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 CancelledError handling in done callbacks of cog init tasks #6203

Conversation

Jackenmen
Copy link
Member

Description of the changes

Stops the done callback from raising on CancelledError, returning silently instead.

Have the changes in this PR been tested?

Yes

@Jackenmen Jackenmen added the Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. label Jul 13, 2023
@Jackenmen Jackenmen added this to the 3.5.3 milestone Jul 13, 2023
@github-actions github-actions bot added Category: Cogs - Downloader This is related to the Downloader cog. Category: Cogs - Mutes This is related to the Mutes cog. labels Jul 13, 2023
@Jackenmen Jackenmen changed the title Fix handling of CancelledError in done callbacks of cog init tasks Fix CancelledError handling in done callbacks of cog init tasks Jul 19, 2023
@Jackenmen Jackenmen merged commit d8e584b into Cog-Creators:V3/develop Jul 19, 2023
@red-githubbot red-githubbot bot added the Changelog Entry: Pending Changelog entry for this PR hasn't been added by repo maintainers yet. label Jul 19, 2023
@Jackenmen Jackenmen deleted the fix_handling_of_cancellederror_in_done_callbacks branch July 19, 2023 20:47
@Jackenmen Jackenmen added Changelog Entry: Added Changelog entry for this PR has already been added to changelog PR. and removed Changelog Entry: Pending Changelog entry for this PR hasn't been added by repo maintainers yet. labels Jul 24, 2023
Dav-Git pushed a commit to Dav-Git/Red-DiscordBot that referenced this pull request Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Cogs - Downloader This is related to the Downloader cog. Category: Cogs - Mutes This is related to the Mutes cog. Changelog Entry: Added Changelog entry for this PR has already been added to changelog PR. 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.

2 participants