aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Scheduler: document coroutine closing elsewhereGravatar MarkKoz2020-07-06-4/+12
|
* Scheduler: assert the coroutine hasn't been awaited yetGravatar MarkKoz2020-07-06-0/+3
| | | | | | It'd fail to schedule the coroutine otherwise anyway. There is also the potential to close the coroutine, which may be unexpected to see for a coroutine that was already running (despite being documented).
* Scheduler: more verbose logging in _await_laterGravatar MarkKoz2020-06-30-7/+10
| | | | | | | | | Showing the task ID in the logs makes them distinguishable from logs for other tasks. The coroutine state is logged because it may come in handy while debugging; the coroutine inspection check hasn't been proven yet in production.
* Scheduler: explain the name param in the docstringGravatar MarkKoz2020-06-29-0/+4
|
* Scheduler: add details to class docstringGravatar MarkKoz2020-06-29-1/+11
|
* Scheduler: only close unawaited coroutinesGravatar MarkKoz2020-06-25-2/+6
| | | | | | The coroutine may cancel the scheduled task, which would also trigger the finally block. The coroutine isn't necessarily finished when it cancels the task, so it shouldn't be closed in this case.
* Scheduler: remove duplicate dict deleteGravatar MarkKoz2020-06-24-1/+0
| | | | | The task is already popped from the dict, so there is no need to delete it afterwards.
* Scheduler: close coroutine if task ID already existsGravatar MarkKoz2020-06-23-1/+7
| | | | This prevents unawaited coroutine warnings.
* Update InfractionScheduler's scheduler to the new APIGravatar MarkKoz2020-06-20-17/+12
|
* Update HelpChannels's scheduler to the new APIGravatar MarkKoz2020-06-20-50/+20
|
* Update Silence's scheduler to the new APIGravatar MarkKoz2020-06-20-26/+6
|
* Update Reminders's scheduler to the new APIGravatar MarkKoz2020-06-20-14/+15
|
* Update Filtering's scheduler to the new APIGravatar MarkKoz2020-06-20-10/+6
|
* Scheduler: add a method to schedule at a specific datetimeGravatar MarkKoz2020-06-20-0/+13
|
* Scheduler: add a method to schedule with a delayGravatar MarkKoz2020-06-20-0/+19
|
* Scheduler: rename "task" param to "coroutine"Gravatar MarkKoz2020-06-20-3/+3
| | | | | Naming it "task" is inaccurate because `create_task` accepts a coroutine rather than a Task. What it does is wrap the coroutine in a Task.
* Scheduler: drop _task suffix from method namesGravatar MarkKoz2020-06-20-3/+3
| | | | It's redundant. After all, this scheduler cannot schedule anything else.
* Scheduler: remove ignore_missing paramGravatar MarkKoz2020-06-20-10/+5
| | | | | The ability to use the `in` operator makes this obsolete. Callers can check themselves if a task exists before they try to cancel it.
* Scheduler: use pop instead of get when cancellingGravatar MarkKoz2020-06-19-7/+7
|
* Scheduler: add support for in operatorGravatar MarkKoz2020-06-19-0/+4
|
* Scheduler: name tasksGravatar MarkKoz2020-06-19-1/+1
| | | | Makes them easier to identify when debugging.
* Scheduler: directly take the awaitable to scheduleGravatar MarkKoz2020-06-19-23/+4
| | | | | | This is a major change which simplifies the interface. It removes the need to implement an abstract method, which means the class can now be instantiated rather than subclassed.
* Scheduler: use separate logger for each instanceGravatar MarkKoz2020-06-19-24/+17
| | | | | | Each instance now requires a name to be specified, which will be used as the suffix of the logger's name. This removes the need to manually prepend every log message with the name.
* Merge pull request #1009 from python-discord/bug/mod/bot-2a/webhook-clydeGravatar kwzrd2020-06-20-8/+57
|\ | | | | Fix 400 when "clyde" is in any webhook username
| * Merge branch 'master' into bug/mod/bot-2a/webhook-clydeGravatar kwzrd2020-06-20-100/+285
| |\ | |/ |/|
* | Merge pull request #1015 from python-discord/kwzrd/pipenv-html-scriptGravatar Mark2020-06-19-0/+1
|\ \ | | | | | | Add Pipenv script for html coverage report
| * | Pipenv: add script for html coverage reportGravatar kwzrd2020-06-19-0/+1
| | | | | | | | | | | | | | | Similarly to the `report` script, this removes the need to invoke coverage when generating the html report.
* | | Merge pull request #1003 from ↵Gravatar Dennis Pham2020-06-19-16/+47
|\ \ \ | |/ / |/| | | | | | | | python-discord/bug/filters/bot-58/removers-ignore-dms Ignore DMs in token and webhook removers
| * | Merge branch 'master' into bug/filters/bot-58/removers-ignore-dmsGravatar Dennis Pham2020-06-19-85/+238
| |\ \ | |/ / |/| |
* | | Merge pull request #892 from Akarys42/#364-offensive-msg-autodeletionGravatar Joseph Banks2020-06-17-9/+80
|\ \ \ | | | | | | | | #364 offensive msg autodeletion
| * | | Update deletion scheduler to use latest watchlist configurationGravatar Joseph Banks2020-06-17-17/+5
| | | |
| * | | Merge branch 'master' into #364-offensive-msg-autodeletionGravatar Joseph Banks2020-06-17-2517/+7259
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #991 from crazygmr101/feature/cooldown-tagGravatar Mark2020-06-17-0/+20
|\ \ \ \ | | | | | | | | | | Create cooldown.md
| * | | | Update bot/resources/tags/customcooldown.mdGravatar Daniel Nash2020-06-15-3/+1
| | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| * | | | Merge branch 'master' into feature/cooldown-tagGravatar Joseph Banks2020-06-12-76/+179
| |\ \ \ \
| * | | | | Add scheme to URLGravatar Daniel Nash2020-06-12-1/+1
| | | | | |
| * | | | | Update customcooldown.mdGravatar Daniel Nash2020-06-12-0/+2
| | | | | |
| * | | | | Make title more specificGravatar Daniel Nash2020-06-12-1/+1
| | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| * | | | | Update bot/resources/tags/customcooldown.mdGravatar Daniel Nash2020-06-10-3/+1
| | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]>
| * | | | | change _cd to message_cooldown Gravatar Daniel Nash2020-06-10-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apply suggestions from code review Co-authored-by: Joseph Banks <[email protected]>
| * | | | | Move the not rate-limited message into elseGravatar Daniel Nash2020-06-10-1/+3
| | | | | |
| * | | | | Update example to not be in a cogGravatar Daniel Nash2020-06-10-12/+10
| | | | | |
| * | | | | Rename to customcooldown.mdGravatar Daniel Nash2020-06-10-0/+0
| | | | | |
| * | | | | Update cooldown.mdGravatar Daniel Nash2020-06-07-1/+1
| | | | | |
| * | | | | Create cooldown.mdGravatar Daniel Nash2020-06-07-0/+22
| | | | | |
* | | | | | Merge pull request #1001 from python-discord/bug/mod/bot-68/ban-strips-noneGravatar Mark2020-06-17-22/+38
|\ \ \ \ \ \ | | | | | | | | | | | | | | Check infraction reason isn't None before shortening it
| * | | | | | Format parameters with a more consistent styleGravatar MarkKoz2020-06-16-19/+19
| | | | | | |
| * | | | | | Add optional type annotations to reason in pardon funcsGravatar MarkKoz2020-06-16-2/+2
| | | | | | |
| * | | | | | Revise inaccurate typehint for Optional reasonGravatar kosayoda2020-06-16-18/+31
| | | | | | |
| * | | | | | Fix check for bot actor in infractionsGravatar MarkKoz2020-06-14-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reason None check should be nested to avoid affecting the else/elif statements that follow.