| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Scheduler: document coroutine closing elsewhere | 2020-07-06 | -4/+12 | |
| | | ||||
| * | Scheduler: assert the coroutine hasn't been awaited yet | 2020-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_later | 2020-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 docstring | 2020-06-29 | -0/+4 | |
| | | ||||
| * | Scheduler: add details to class docstring | 2020-06-29 | -1/+11 | |
| | | ||||
| * | Scheduler: only close unawaited coroutines | 2020-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 delete | 2020-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 exists | 2020-06-23 | -1/+7 | |
| | | | | | This prevents unawaited coroutine warnings. | |||
| * | Update InfractionScheduler's scheduler to the new API | 2020-06-20 | -17/+12 | |
| | | ||||
| * | Update HelpChannels's scheduler to the new API | 2020-06-20 | -50/+20 | |
| | | ||||
| * | Update Silence's scheduler to the new API | 2020-06-20 | -26/+6 | |
| | | ||||
| * | Update Reminders's scheduler to the new API | 2020-06-20 | -14/+15 | |
| | | ||||
| * | Update Filtering's scheduler to the new API | 2020-06-20 | -10/+6 | |
| | | ||||
| * | Scheduler: add a method to schedule at a specific datetime | 2020-06-20 | -0/+13 | |
| | | ||||
| * | Scheduler: add a method to schedule with a delay | 2020-06-20 | -0/+19 | |
| | | ||||
| * | Scheduler: rename "task" param to "coroutine" | 2020-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 names | 2020-06-20 | -3/+3 | |
| | | | | | It's redundant. After all, this scheduler cannot schedule anything else. | |||
| * | Scheduler: remove ignore_missing param | 2020-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 cancelling | 2020-06-19 | -7/+7 | |
| | | ||||
| * | Scheduler: add support for in operator | 2020-06-19 | -0/+4 | |
| | | ||||
| * | Scheduler: name tasks | 2020-06-19 | -1/+1 | |
| | | | | | Makes them easier to identify when debugging. | |||
| * | Scheduler: directly take the awaitable to schedule | 2020-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 instance | 2020-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-clyde | 2020-06-20 | -8/+57 | |
| |\ | | | | | Fix 400 when "clyde" is in any webhook username | |||
| | * | Merge branch 'master' into bug/mod/bot-2a/webhook-clyde | 2020-06-20 | -100/+285 | |
| | |\ | |/ |/| | ||||
| * | | Merge pull request #1015 from python-discord/kwzrd/pipenv-html-script | 2020-06-19 | -0/+1 | |
| |\ \ | | | | | | | Add Pipenv script for html coverage report | |||
| | * | | Pipenv: add script for html coverage report | 2020-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 ↵ | 2020-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-dms | 2020-06-19 | -85/+238 | |
| | |\ \ | |/ / |/| | | ||||
| * | | | Merge pull request #892 from Akarys42/#364-offensive-msg-autodeletion | 2020-06-17 | -9/+80 | |
| |\ \ \ | | | | | | | | | #364 offensive msg autodeletion | |||
| | * | | | Update deletion scheduler to use latest watchlist configuration | 2020-06-17 | -17/+5 | |
| | | | | | ||||
| | * | | | Merge branch 'master' into #364-offensive-msg-autodeletion | 2020-06-17 | -2517/+7259 | |
| | |\ \ \ | |/ / / |/| | | | ||||
| * | | | | Merge pull request #991 from crazygmr101/feature/cooldown-tag | 2020-06-17 | -0/+20 | |
| |\ \ \ \ | | | | | | | | | | | Create cooldown.md | |||
| | * | | | | Update bot/resources/tags/customcooldown.md | 2020-06-15 | -3/+1 | |
| | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]> | |||
| | * | | | | Merge branch 'master' into feature/cooldown-tag | 2020-06-12 | -76/+179 | |
| | |\ \ \ \ | ||||
| | * | | | | | Add scheme to URL | 2020-06-12 | -1/+1 | |
| | | | | | | | ||||
| | * | | | | | Update customcooldown.md | 2020-06-12 | -0/+2 | |
| | | | | | | | ||||
| | * | | | | | Make title more specific | 2020-06-12 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]> | |||
| | * | | | | | Update bot/resources/tags/customcooldown.md | 2020-06-10 | -3/+1 | |
| | | | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]> | |||
| | * | | | | | change _cd to message_cooldown | 2020-06-10 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apply suggestions from code review Co-authored-by: Joseph Banks <[email protected]> | |||
| | * | | | | | Move the not rate-limited message into else | 2020-06-10 | -1/+3 | |
| | | | | | | | ||||
| | * | | | | | Update example to not be in a cog | 2020-06-10 | -12/+10 | |
| | | | | | | | ||||
| | * | | | | | Rename to customcooldown.md | 2020-06-10 | -0/+0 | |
| | | | | | | | ||||
| | * | | | | | Update cooldown.md | 2020-06-07 | -1/+1 | |
| | | | | | | | ||||
| | * | | | | | Create cooldown.md | 2020-06-07 | -0/+22 | |
| | | | | | | | ||||
| * | | | | | | Merge pull request #1001 from python-discord/bug/mod/bot-68/ban-strips-none | 2020-06-17 | -22/+38 | |
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | Check infraction reason isn't None before shortening it | |||
| | * | | | | | | Format parameters with a more consistent style | 2020-06-16 | -19/+19 | |
| | | | | | | | | ||||
| | * | | | | | | Add optional type annotations to reason in pardon funcs | 2020-06-16 | -2/+2 | |
| | | | | | | | | ||||
| | * | | | | | | Revise inaccurate typehint for Optional reason | 2020-06-16 | -18/+31 | |
| | | | | | | | | ||||
| | * | | | | | | Fix check for bot actor in infractions | 2020-06-14 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reason None check should be nested to avoid affecting the else/elif statements that follow. | |||