aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Document new kwarg in docstringGravatar kosayoda2020-07-15-0/+2
|
* Merge branch 'master' into talentpool-oldestGravatar Dennis Pham2020-07-14-55/+143
|\
| * Merge pull request #1032 from slushiegoose/sanitize-bot-outputGravatar Mark2020-07-14-53/+141
| |\ | | | | | | Sanitize bot output
| | * fix misaligned indentationGravatar Slushie2020-07-14-37/+37
| | |
| | * edit snekbox tests to work with filteringGravatar Slushie2020-07-13-0/+12
| | |
| | * check for the filter_cog in case it is unloadedGravatar Slushie2020-07-13-1/+3
| | |
| | * rename the `_filter_eval` function to be a public functionGravatar Slushie2020-07-13-2/+2
| | |
| | * Add missing variable assignment to stop NameErrors occurringGravatar Slushie2020-07-09-0/+1
| | |
| | * Fix missing hypenGravatar slushiegoose2020-07-09-1/+1
| | | | | | | | | Co-authored-by: Leon Sandøy <[email protected]>
| | * Call the filter eval command after receiving an eval resultGravatar Slushie2020-07-08-1/+6
| | |
| | * Create a filtering function to filter eval resultsGravatar Slushie2020-07-08-52/+120
| | |
| | * Edit BadArgument error messageGravatar Slushie2020-07-08-1/+1
| | |
| * | Remove redundant kwarg in !kick and !shadow_kickGravatar kosayoda2020-07-14-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | The kwarg `active=False` is already being passed in `apply_kick`, therefore passing it in the parent callers result in a TypeError. Fixes #976 Fixes BOT-5P
* | | Achieve feature parity with talentpoolGravatar kosayoda2020-07-14-2/+17
| | |
* | | Add command to list nominees by oldest firstGravatar kosayoda2020-07-14-0/+11
| | |
* | | Accept argument to order nominees by oldest firstGravatar kosayoda2020-07-14-2/+6
| | |
* | | Pass argument as kwarg to preserve functionalityGravatar kosayoda2020-07-14-1/+1
| | |
* | | Allow ordering watched users by oldest firstGravatar kosayoda2020-07-14-1/+7
|/ /
* | Whitelisting some popular communitiesGravatar Leon Sandøy2020-07-13-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following communities are whitelisted by this commit: - Django - Programming Discussions - JetBrains Community - Raspberry Pi - Programmers Hangout - SpeakJS - DevCord - Unity - Programmer Humor - Microsoft Community Most of these are partners, or otherwise friendly communities that aren't worth pinging mods over.
* | Merge pull request #1039 from ↵Gravatar Dennis Pham2020-07-13-4/+16
|\ \ | | | | | | | | | | | | python-discord/1038_allow_role_mentions_in_specific_areas Allow role mentions in specific areas
| * \ Merge branch 'master' into 1038_allow_role_mentions_in_specific_areasGravatar Dennis Pham2020-07-13-169/+153
| |\ \ | |/ / |/| |
* | | Fix rescheduling of edited infractionsGravatar MarkKoz2020-07-12-1/+1
| | | | | | | | | | | | | | | | | | | | | It was attempting to schedule a dictionary instead of a coroutine. Fixes #1043 Fixes BOT-6Y
* | | Merge pull request #1017 from python-discord/feat/backend/800/scheduler-redesignGravatar Mark2020-07-12-169/+153
|\ \ \ | | | | | | | | Scheduler Redesign
| * \ \ Merge branch 'master' into feat/backend/800/scheduler-redesignGravatar Mark2020-07-12-217/+683
| |\ \ \ | |/ / / |/| | |
| * | | 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.
| | * | Revert "Ping @Moderators in ModLog"Gravatar Leon Sandøy2020-07-13-15/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's continue to use "@everyone" for now, and add an explicit allow for it so that it successfully pings people. There's a full justification for this in the pull request. https://github.com/python-discord/bot/issues/1038
| | * | Remove pointless comment Gravatar Leon Sandøy2020-07-12-1/+1
| | | | | | | | | | | | | | | | | | | | This comment violates the DRY principle. Co-authored-by: Sebastiaan Zeeff <[email protected]>
| | * | Allow role pings in Syncers and help_channels.pyGravatar Leon Sandøy2020-07-12-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we're running Discord 1.4.0a, we need to explicitely allow all the role mentions for sends that don't use ping one of the globally whitelisted role pings, which are Moderators, Admins and Owners. We were pinging roles other than Mods+ in exactly two cases: - Inside the Syncers, whenever we ask for sync confirmation (if the number of roles or users to sync is unusually high) - In the help_channels.py system, whenever we max out help channels and are unable to create more. This commit addresses both of these. GitHub #1038 https://github.com/python-discord/bot/issues/1038