| Commit message (Collapse) | Author | Age | Lines |
| | |
|
| |\ |
|
| | |\
| | |
| | | |
Sanitize bot output
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | | |
Co-authored-by: Leon Sandøy <[email protected]>
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| |/ / |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |\ \
| | |
| | |
| | |
| | | |
python-discord/1038_allow_role_mentions_in_specific_areas
Allow role mentions in specific areas
|
| | |\ \
| |/ /
|/| | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
It was attempting to schedule a dictionary instead of a coroutine.
Fixes #1043
Fixes BOT-6Y
|
| |\ \ \
| | | |
| | | | |
Scheduler Redesign
|
| | |\ \ \
| |/ / /
|/| | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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).
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
The task is already popped from the dict, so there is no need to delete
it afterwards.
|
| | | | |
| | | |
| | | |
| | | | |
This prevents unawaited coroutine warnings.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
| | | |
| | | |
| | | | |
It's redundant. After all, this scheduler cannot schedule anything else.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
The ability to use the `in` operator makes this obsolete. Callers can
check themselves if a task exists before they try to cancel it.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Makes them easier to identify when debugging.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This comment violates the DRY principle.
Co-authored-by: Sebastiaan Zeeff <[email protected]>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|