Skip to content

Use scheduling create_task util to create tasks#1825

Merged
Xithrius merged 3 commits into
python-discord:mainfrom
Numerlor:scheduling-create_task-init
Sep 26, 2021
Merged

Use scheduling create_task util to create tasks#1825
Xithrius merged 3 commits into
python-discord:mainfrom
Numerlor:scheduling-create_task-init

Conversation

@Numerlor
Copy link
Copy Markdown
Contributor

closes #1807
replaces loop.create_task and asyncio.create_task usages with the scheduling util so exception logging is attached to created tasks

The util attaches an error logging callback instead of relying on python's
exception logging which only occurs when the task is destroyed
The util attaches an error logging callback instead of relying on python's
exception logging which only occurs when the task is destroyed
Comment thread bot/exts/help_channels/_cog.py
Comment thread bot/exts/moderation/defcon.py
Comment thread bot/exts/utils/reminders.py
@Xithrius Xithrius added a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) p: 2 - normal Normal Priority s: needs review Author is waiting for someone to review and approve t: enhancement Changes or improvements to existing features labels Sep 11, 2021
@Xithrius Xithrius enabled auto-merge September 26, 2021 19:21
@Xithrius Xithrius merged commit e98719a into python-discord:main Sep 26, 2021
@Numerlor Numerlor deleted the scheduling-create_task-init branch September 26, 2021 19:25
@Xithrius Xithrius removed the s: needs review Author is waiting for someone to review and approve label Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) p: 2 - normal Normal Priority t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace bot.loop.create_task usages with scheduling.create_task

3 participants