aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | | | | | Merge branch 'master' into bug/filters/bot-58/removers-ignore-dmsGravatar Dennis Pham2020-06-19-85/+238
| | |\ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | |
| | * | | | | | | | Webhook remover: exit early if message already deletedGravatar MarkKoz2020-06-15-2/+8
| | | | | | | | | |
| | * | | | | | | | Webhook remover: ignore DMs and bot messagesGravatar MarkKoz2020-06-15-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Can't remove messages in DMs, so don't bother trying.
| | * | | | | | | | Token remover: exit early if message already deletedGravatar MarkKoz2020-06-15-3/+22
| | | | | | | | | |
| | * | | | | | | | Token remover: move bot check to on_messageGravatar MarkKoz2020-06-15-21/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It just makes more sense to me to filter out messages at an earlier stage.
| | * | | | | | | | Token remover: ignore DMsGravatar MarkKoz2020-06-15-0/+13
| | | |_|_|/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's a private channel so there's no risk of a token "leaking". Furthermore, messages cannot be deleted in DMs.
| | | * | | | | | Write unit test for `sub_clyde`Gravatar kwzrd2020-06-20-0/+27
| | | | | | | | |
| | | * | | | | | Preserve empty string when substituting clydeGravatar MarkKoz2020-06-19-0/+2
| | | | | | | | |
| | | * | | | | | Make sub_clyde case-sensitive and use Cyrillic e'sGravatar MarkKoz2020-06-17-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Cyrillic characters are more likely to be rendered similarly to their Latin counterparts than the math sans-serif characters.
| | | * | | | | | Fix 400 when "clyde" is in webhook usernameGravatar MarkKoz2020-06-15-8/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Discord just disallows this name.
* | | | | | | | | Incidents tests: remove unnecessary patchGravatar kwzrd2020-06-20-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is already being patched at class-level.
* | | | | | | | | Incidents: try-except Signal creationGravatar kwzrd2020-06-20-11/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Suggested by Mark during review. This follows the "ask for forgiveness rather than permission" paradigm, ends up being less code to read, and may be seen as more logical / safer. The `ALLOWED_EMOJI` set was renamed to `ALL_SIGNALS` as this now better communicates the set's purpose. Tests adjusted as appropriate. Co-authored-by: MarkKoz <[email protected]>
* | | | | | | | | Incidents: simplify set operation in `has_signals`Gravatar kwzrd2020-06-20-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using `issubset` is a much simpler & more readable way of expressing the relationship between the two sets. Suggested by Mark during review. Co-authored-by: MarkKoz <[email protected]>
* | | | | | | | | Incidents: make crawl limit & sleep module-level constantsGravatar kwzrd2020-06-20-10/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Requested during review. Co-authored-by: ks129 <[email protected]> Co-authored-by: Joseph Banks <[email protected]>
* | | | | | | | | Incidents: make logs contain the message id they pertain toGravatar kwzrd2020-06-20-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Suggested by Mark during review. Co-authored-by: MarkKoz <[email protected]>
* | | | | | | | | Incidents: remove redundant `exc_info` passingGravatar kwzrd2020-06-20-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pointed out by Mark during review that this is unnecessary, as logging using `exception` automatically appends the `exc_info` of the handled exception when done in an except block. Co-authored-by: MarkKoz <[email protected]>
* | | | | | | | | Incidents: reduce excessive whitespaceGravatar kwzrd2020-06-19-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is way too spacious for how little is happening here. Suggested by ks129. Co-authored-by: ks129 <[email protected]>
* | | | | | | | | Incidents: annotate possible None typeGravatar kwzrd2020-06-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Caught during review by ks129. Co-authored-by: ks129 <[email protected]>
* | | | | | | | | Incidents: remove broad noqaGravatar kwzrd2020-06-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was originally in place to silence a PyCharm warning regarding accessing the private attributes. However, since there is no corresponding error code to specify, the noqa will silence any linter warning, which is potentially dangerous, and seems to be bad practice.
* | | | | | | | | Incidents: review log levels; use `trace` where appropriateGravatar kwzrd2020-06-19-19/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Logs useful when observing the internals but too verbose for DEBUG are reduced to TRACE.
* | | | | | | | | Incidents tests: write tests for `on_raw_reaction_add`Gravatar kwzrd2020-06-19-0/+128
| | | | | | | | |
* | | | | | | | | Incidents tests: write tests for `crawl_incidents`Gravatar kwzrd2020-06-19-0/+58
| | | | | | | | |
* | | | | | | | | Incidents tests: add helper for mocking async for-loopsGravatar kwzrd2020-06-19-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See the docstring. This does not make the ambition to be powerful enough to be included in `tests.helpers`, and is only intended for local purposes.
* | | | | | | | | Merge branch 'origin/master' into kwzrd/incidentsGravatar kwzrd2020-06-19-251/+643
|\| | | | | | | |
| * | | | | | | | 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.
| | * | | | | | | | | | | Check infraction reason isn't None before shortening itGravatar MarkKoz2020-06-14-5/+8
| | | |_|_|/ / / / / / / | | |/| | | | | | | | |
| * | | | | | | | | | | Add LMGTFY to domain blacklistGravatar Joseph Banks2020-06-17-0/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Merge pull request #987 from python-discord/help_channel_rediscacheGravatar Kieran Siek2020-06-16-53/+98
| |\ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|/ / / / / / | |/| | | | | | | | | | Add persistence to the help channel system
| | * | | | | | | | | | Merge branch 'master' into help_channel_rediscacheGravatar Kieran Siek2020-06-16-243/+585
| | |\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / | |/| | | | | | | | | |
| * | | | | | | | | | | Log exception info for failed attachment uploadsGravatar MarkKoz2020-06-15-1/+1
| | |/ / / / / / / / / | |/| | | | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-31 06:37:00 +0000