aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | | | | | | | | | | | | Use pre-commit in pipenv lint scriptGravatar MarkKoz2020-03-03-4/+12
| | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | Add more pre-commit hooksGravatar MarkKoz2020-03-02-10/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove trailing whitespaces * Specify error code for a noqa in the free command
| | | * | | | | | | | | | | | | | Use raw strings for docstrings with forward slashesGravatar Sebastiaan Zeeff2020-03-04-3/+3
| | |/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A few docstrings in `bot.cogs.extensions` have forward slashed in them to escape Markdown rendering when our help feature uses these docstring in a Discord message. However, the use of forward slashes with an invalid escape sequence in docstrings now raises a DeprecationWarning in Python: /home/sebastiaan/pydis/repositories/bot/bot/cogs/extensions.py:72: DeprecationWarning: invalid escape sequence \* PEP 257 (Docstring Conventions, https://www.python.org/dev/peps/pep-0257/) states that raw strings should be used for docstrings that use forward slashes, so I've added the `r`-prefix to the docstrings that use forward slashes.
| * | | | | | | | | | | | | | | Merge pull request #815 from python-discord/watchchannel-dmGravatar scragly2020-03-04-2/+9
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | Prevent exception if a watched user sends a DM to the bot
| | * | | | | | | | | | | | | | Prevent exception if a watched user sends a DM to the botGravatar S. Co12020-03-03-2/+9
| |/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous embed assumed that the messages would be sent on the server, where the channel would have a name and the message would have a jump URL. For a DM, neither of these are present and an exception will be raised when attempting to construct the embed for the webhook to send.
| | | * | | | | | | | | | | | Merge branch 'master' into antimalware-loggingGravatar S. Co12020-03-03-1076/+1451
| | | |\ \ \ \ \ \ \ \ \ \ \ \ | | |_|/ / / / / / / / / / / / | |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | | Merge pull request #750 from python-discord/bug/backend/b748/resolver-in-coroGravatar Mark2020-03-03-31/+93
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | Create AsyncResolver inside a coroutine to avoid DeprecationWarning
| | * | | | | | | | | | | | | Merge branch 'master' into bug/backend/b748/resolver-in-coroGravatar Mark2020-03-03-1048/+1367
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / | |/| | | | | | | | | | | | |
| | * | | | | | | | | | | | | Fix typo in commentGravatar Mark2020-03-03-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Leon Sandøy <[email protected]>
| | * | | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-25-1156/+2657
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug/backend/b748/resolver-in-coro
| | * | | | | | | | | | | | | | API: close existing open session before recreating itGravatar MarkKoz2020-02-15-3/+4
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | API: add argument to force recreation of the sessionGravatar MarkKoz2020-02-15-4/+10
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Bot: send not-closed warnings as log messagesGravatar MarkKoz2020-02-15-12/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Real" warnings weren't showing up for some reason.
| | * | | | | | | | | | | | | | Bot: warn when connector/session not closed when recreatingGravatar MarkKoz2020-02-15-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aiohttp does warn too, but these warnings will provide more immediate feedback.
| | * | | | | | | | | | | | | | Bot: call _recreate() in clear()Gravatar MarkKoz2020-02-15-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because discord.py recreates the HTTPClient session, may as well follow suite and recreate our own stuff here too.
| | * | | | | | | | | | | | | | Bot: move connector/session recreation to a separate functionGravatar MarkKoz2020-02-15-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function itself doesn't need to be a coroutine. It just has to be called in a coroutine (or, more indirectly, in an async context?).
| | * | | | | | | | | | | | | | Bot: override login() instead of start()Gravatar MarkKoz2020-02-15-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The client can be used without running a bot so it makes more sense for the connector to be created when logging in, which is done in both cases, rather than in start(), which is only used when running a bot.
| | * | | | | | | | | | | | | | Bot: fix error trying to close a None sessionGravatar MarkKoz2020-02-14-1/+3
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Bot: add warning for when connector is a specified kwargGravatar MarkKoz2020-02-13-0/+6
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Bot: avoid DeprecationWarning for aiohttp.AsyncResolver (fix #748)Gravatar MarkKoz2020-02-13-15/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AsyncResolver has to be created inside a coroutine so it's moved inside start(). Consequently, the APIClient session is also recreated inside start() now. When using clear(), the default connector is used for the session it recreates because clear() is not a coroutine. This should only affect requests made to the Discord API via the Client when not using it to run a bot; starting the bot will re-create the session with the custom connector. * Close connector and resolver when bot closes
| | * | | | | | | | | | | | | | API: accept additional session kwargs for recreate()Gravatar MarkKoz2020-02-13-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These kwargs are merged with the kwargs given when the APIClient was created. This is useful for facilitating changing the session's connector with a new instance when the session needs to be recreated. * Rename _session_args attribute to _default_session_kwargs
| | | | * | | | | | | | | | | | Add logging to antimalware cog & expand user feedbackGravatar S. Co12020-03-01-17/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add generic handling for multi-file uploads * Log user, id, and blocked extensions * Provide the full list of attachment filenames as a logging extra * Provide feedback on all blacklisted file types uploaded
| | | | | | * | | | | | | | | | Merge remote-tracking branch 'origin/master' into zen-commandGravatar MarkKoz2019-12-12-1209/+3144
| | | | | | |\ \ \ \ \ \ \ \ \ \
| | | | | | * | | | | | | | | | | apply kosa's requested changes.Gravatar Kingsley McDonald2019-11-02-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - return None from the command's coroutine as hinted, rather than a discord.Message object. - only check for one negative sign on !zen index searches (rather than any amount) so that `int(...)` does not fail. - provide a range of valid indices when a user requests a !zen index out of range.
| | | | | | * | | | | | | | | | | implement !zen command.Gravatar Kingsley McDonald2019-11-01-2/+96
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Mod Utils): Removed unnecessary `textwrap` importGravatar Karlis S2020-03-13-1/+0
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Mod Utils + Tests): Moved constants from tests to utils, applied changeGravatar Karlis S2020-03-13-34/+34
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed Pardon Color constant.Gravatar Karlis S2020-03-13-3/+1
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed Infraction Color constant.Gravatar Karlis S2020-03-13-5/+4
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Fixed formatting in `notify_pardon` test.Gravatar Karlis S2020-03-13-1/+2
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Moved `return_value` to `patch` decorator.Gravatar Karlis S2020-03-13-4/+2
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed unnecessary mock resetting.Gravatar Karlis S2020-03-13-2/+0
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Added 2 call check to `post_infraction` test.Gravatar Karlis S2020-03-13-2/+2
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Revise docstrings for moderation util testsGravatar MarkKoz2020-03-12-10/+16
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed `once` from `post_infraction` test due ↵Gravatar Karlis S2020-03-12-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tests failing.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Added mock reset due fail.Gravatar ks1232020-03-12-0/+2
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Added call check for `ctx.send` in `post_user` test.Gravatar ks1232020-03-12-0/+1
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Move errors from booleans to actual errors in ↵Gravatar ks1232020-03-12-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `post_user` test.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Hard-coded `self.ctx` argument to `post_user` ↵Gravatar ks1232020-03-12-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test, renamed current `args` to `user`, applied this in code.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Fixed formatting in `notify_infraction` test.Gravatar ks1232020-03-12-3/+1
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed unnecessary symbols from ↵Gravatar ks1232020-03-12-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `has_active_infraction` test `infraction_nr` variable and changes this to more unique number.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Moved embed generating to test cases loop from ↵Gravatar ks1232020-03-12-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test cases listing, added icon to test cases in `notify_pardon` test
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Added params to variable in ↵Gravatar ks1232020-03-12-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `has_active_infraction` test.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Create extra new tests set for `post_infraction` ↵Gravatar ks1232020-03-12-103/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | testing, removed old.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Moved mock resetting to beginning of subtest in ↵Gravatar ks1232020-03-08-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `post_user` and `send_private_embed` test.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed unnecessary `if` check from ↵Gravatar ks1232020-03-08-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `send_private_embed` test
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Hard-coded args for `send_private_embed` test.Gravatar ks1232020-03-08-10/+6
| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Added new check to `post_user` test (`ctx.send` ↵Gravatar ks1232020-03-08-35/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | content test), improved test cases.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Moved `expected_output` from `Dict` to ↵Gravatar ks1232020-03-08-15/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `discord.Embed` in `notify_pardon` test.
* | | | | | | | | | | | | | | | | (Moderation Utils Tests): Removed unnecessary `ctx.send` `side_effect` from ↵Gravatar ks1232020-03-08-11/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `notify_pardon`, applied changes to test cases.