aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | * | | | | | | | | ModLog: refactor on_member_updateGravatar MarkKoz2020-06-14-23/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Exclude all sequences/mapping types rather than excluding by name * Replace MEMBER_CHANGES_SUPPRESSED with excludes as DeepDiff args * Don't keep track of "done" attributes - there shouldn't be dupes
| | | | * | | | | | | | | ModLog: fix excluded None values in on_member_updateGravatar MarkKoz2020-06-14-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was preventing diffs for added nicknames from showing, among other things.
| | | | * | | | | | | | | ModLog: remove user diff in on_member_updateGravatar MarkKoz2020-06-14-21/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The correct event for user changes is on_user_update, so this code does nothing in the on_member_update event.
| | | | * | | | | | | | | ModLog: fix AttributeError in on_member_updateGravatar MarkKoz2020-06-14-22/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `iterable_item_removed` and `iterable_item_added` lack `new_value` and `old_value`. Instead, they just contain the actual value added or removed. The code was incorrectly trying to access old and new values for the iterable changes. The iterable changes are only useful for the role diff, but they aren't even needed for that. The role diff calculation has been refactored to always get the diff rather than doing it only if it sees there has been a change to the `_roles` attribute. To be clear, `_roles` only has IDs, which is why its diff isn't that useful anyway. To use it, the code would have to get the Role objects, which is basically what the `roles` property already does. `_cs_roles` seems to be some Role object cache, but its reliability is unclear.
* | | | | | | | | | | | | Code block: dedent code before validating itGravatar MarkKoz2020-06-28-4/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If it's indented too far, the AST parser will fail.
* | | | | | | | | | | | | Code block: re-add indentation fixing functionGravatar MarkKoz2020-06-28-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's still useful to fix indentation to ensure AST is correctly parsed. This function deals with the relatively common case of a the leading spaces of the first line being left out when copy-pasting.
* | | | | | | | | | | | | Code block: support IPython REPL detectionGravatar MarkKoz2020-06-27-6/+53
| | | | | | | | | | | | |
* | | | | | | | | | | | | Code block: fix BadLanguage creationGravatar MarkKoz2020-06-27-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Forgot to change the kwarg names when the attributes were renamed.
* | | | | | | | | | | | | HelpChannels: fix is_in_category callGravatar MarkKoz2020-06-27-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was still using it like it was a method of the class rather than calling it from the channel utils module.
* | | | | | | | | | | | | Code block: clarify get_instructions's docstringGravatar MarkKoz2020-06-27-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It wasn't clear that it also parses the message content.
* | | | | | | | | | | | | Code block: simplify channel cooldown dict creationGravatar MarkKoz2020-06-27-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Code block: use config constant for cooldownGravatar MarkKoz2020-06-27-2/+4
| | | | | | | | | | | | |
* | | | | | | | | | | | | Code block: make PY_LANG_CODES more visibleGravatar MarkKoz2020-06-27-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The declaration was a bit hidden between the two regular expressions.
* | | | | | | | | | | | | Code block: rename BadLanguage attributesGravatar MarkKoz2020-06-27-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `has_` prefix it clarifies that they're booleans. Co-authored-by: Numerlor <[email protected]>
* | | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-06-22-133/+469
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug/utility/829/non-py-code-detection
| * | | | | | | | | | | | Merge pull request #1002 from ↵Gravatar kwzrd2020-06-22-23/+93
| |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/backend/bot-50/sync-ignore-guilds Sync: ignore events from other guilds
| | * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug/backend/bot-50/sync-ignore-guildsGravatar kwzrd2020-06-22-109/+375
| | |\ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / | |/| | | | | | | | | | | |
| * | | | | | | | | | | | | Merge pull request #950 from ks129/logging-testsGravatar Kieran Siek2020-06-22-0/+32
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | Created tests for startup logging
| | * | | | | | | | | | | | Merge branch 'master' into logging-testsGravatar Kieran Siek2020-06-22-790/+2732
| | |\ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / | |/| | | | | | | | | | | |
| * | | | | | | | | | | | | 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.
| | * | | | | | | | | | | | | | | | 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
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | |