aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge branch 'master' into spoiler-checkGravatar Leon Sandøy2020-02-29-297/+857
|\
| * Merge pull request #710 from python-discord/eval-enhancementsGravatar Mark2020-02-28-32/+479
| |\ | | | | | | Eval cog improvements
| | * Snekbox: mention re-evaluation feature in the command's docstringGravatar MarkKoz2020-02-28-1/+4
| | |
| | * Merge remote-tracking branch 'origin/master' into eval-enhancementsGravatar MarkKoz2020-02-28-1440/+3084
| | |\ | | |/ | |/|
| * | Merge pull request #757 from python-discord/feat/backend/b131/error-handlingGravatar Leon Sandøy2020-02-28-102/+149
| |\ \ | | | | | | | | Handle more types of CommandError
| | * \ Merge branch 'master' into feat/backend/b131/error-handlingGravatar Leon Sandøy2020-02-28-196/+294
| | |\ \ | | |/ / | |/| |
| * | | Merge pull request #782 from python-discord/feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-163/+229
| |\ \ \ | | | | | | | | | | Rename constants to match their guild counterparts
| | * \ \ Merge branch 'master' into feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-33/+65
| | |\ \ \ | | |/ / / | |/| | |
| | * | | Fix mismatched constant names in syncer testsGravatar S. Co12020-02-25-6/+6
| | | | |
| | * | | Configure staff role & channel groupings in YAMLGravatar S. Co12020-02-25-10/+27
| | | | | | | | | | | | | | | Delete duplicate keys that were missed in the merge
| | * | | Merge branch 'master' into feat/backend/b496/rename-constantsGravatar S. Co12020-02-25-546/+1676
| | |\ \ \
| | * | | | Tests: fix setting bot-commands ID in information testsGravatar MarkKoz2020-02-23-4/+4
| | | | | |
| | * | | | Constants: rename Guild.Constant.ignored to modlog_blacklistGravatar MarkKoz2020-02-23-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | This name better explains what the list is for.
| | * | | | Config: replace abbreviated lists with normal onesGravatar MarkKoz2020-02-23-4/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lists were getting too long to be readable as one line. Having each element on a separate line also reduces merge conflicts.
| | * | | | Config: add underscore to devcontribGravatar MarkKoz2020-02-23-2/+2
| | | | | |
| | * | | | Config: fix DEV_LOG variable thingyGravatar MarkKoz2020-02-23-2/+2
| | | | | |
| | * | | | Always load doc and verification extensionsGravatar MarkKoz2020-02-23-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They used to only be loaded in "debug mode" because the main guild was used to test the bot. However, we have since moved to using a separate test guild so it's no longer a concern if these cogs get loaded. It was confusing to some contributors as to why these cogs were not being loaded since the debug mode isn't really documented anywhere.
| | * | | | Config: rename channels to match their names in the guildGravatar MarkKoz2020-02-23-76/+76
| | | | | |
| | * | | | Constants: remove code jam champions roleGravatar MarkKoz2020-02-23-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Nothing was using it.
| | * | | | Constants: rename developers role back to verifiedGravatar MarkKoz2020-02-23-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It makes the code which uses it more readable. A comment was added to explain the discrepancy between the constant's name and the name in the guild.
| | * | | | Constants: rename roles to match their names in the guildGravatar MarkKoz2020-02-23-28/+28
| | | | | |
| | * | | | Config: remove checkpoint_test and devtestGravatar MarkKoz2020-02-23-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They no longer exist in the guild. * Move devlog under the "Logs" category
| | * | | | Config: shorten name of PYTHON_COMMUNITY_ROLEGravatar MarkKoz2020-02-23-7/+7
| | | | | |
| | * | | | Config: split channels into categoriesGravatar MarkKoz2020-02-23-21/+39
| | | | | |
| | * | | | Config: split roles into categoriesGravatar MarkKoz2020-02-23-12/+16
| | | | | |
| | * | | | Config: rename roles to match their names in the guildGravatar MarkKoz2020-02-23-18/+18
| | | | | |
| | | | * | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-25-1158/+2657
| | | | |\ \ | | | | |/ / | | | |/| | | | | | | | feat/backend/b131/error-handling
| | | | * | Error handler: handle CommandOnCooldown errorsGravatar MarkKoz2020-02-16-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | Simply send the error's default message to the invoking context.
| | | | * | Error handler: update docstrings to reflect recent changesGravatar MarkKoz2020-02-16-20/+39
| | | | | |
| | | | * | Error handler: simplify check failure handler & handle bot missing rolesGravatar MarkKoz2020-02-16-13/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | discord.py's default error messages are quite descriptive already so there really isn't a need to write our own. Therefore, the log calls were removed so that the generic debug log message is used in the on_command_error. In addition to handling missing bot permissions, missing bot roles are also handled. The message doesn't specify which because it doesn't really matter to the end-user. The logs will use the default error messages as described above, and those will contain the specific roles or permissions that are missing.
| | | | * | Error handler: (almost) always log the error being handledGravatar MarkKoz2020-02-16-16/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The log level is debug for most errors and it's mainly useful for precisely that - debugging. This is why some "useless" errors are also logged e.g. CommandNotFound. Unexpected errors and some API errors will still have higher levels. * Add a single log statement to the end of the handler to cover UserInputError, CheckFailure, and CommandNotFound (when it's not trying to get a tag) * Log 404s from API
| | | | * | Error handler: handle ArgumentParsingErrorGravatar MarkKoz2020-02-16-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Simply send the error message with the help command.
| | | | * | Error handler: handle BadUnionArgumentGravatar MarkKoz2020-02-16-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send a message specifying the parameter name, the converters used, and the last error message from the converters.
| | | | * | Error handler: handle TooManyArgumentsGravatar MarkKoz2020-02-16-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | Send a message specifying the error reason.
| | | | * | Error handler: handle MissingRequiredArgumentGravatar MarkKoz2020-02-16-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | Send a message indicating which argument is missing.
| | | | * | Error handler: simplify error importsGravatar MarkKoz2020-02-16-29/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import the errors module and qualify the error types with it rather than importing a large list of error types.
| | | | * | Error handler: create separate function to handle UserInputErrorGravatar MarkKoz2020-02-16-12/+17
| | | | | |
| | | | * | Error handler: move help command retrieval to a separate functionGravatar MarkKoz2020-02-16-11/+18
| | | | | |
| | | | * | Error handler: move tag retrieval to a separate functionGravatar MarkKoz2020-02-16-18/+28
| | | | | |
| | | | * | Error handler: log unhandled exceptions instead of re-raisingGravatar MarkKoz2020-02-16-2/+2
| | | | | |
| | | | * | Error handler: create separate function to handle ResponseCodeErrorGravatar MarkKoz2020-02-16-14/+17
| | | | | |
| | | | * | Error handler: create separate function to handle CheckFailureGravatar MarkKoz2020-02-16-15/+29
| | | | | |
| | | | * | Moderation: fix member not found error not being shownGravatar MarkKoz2020-02-16-1/+1
| | | | | |
| | | | | * Ignore NotFound errors inside continue_evalGravatar Matteo2020-02-28-24/+25
| | | | | | | | | | | | | | | | | | | | | | | | It could have caused some errors if the user delete his own message
| | | | | * Return only the new code in continue_eval and check for truthiness insteadGravatar Matteo2020-02-28-11/+10
| | | | | |
| | | | | * Use unicode code point instead of literal for the snekbox re-eval emojiGravatar Matteo2020-02-27-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | Unicode literals aren't really safe compared to code points
| | | | | * Split the eval command procedure into two functions.Gravatar Matteo2020-02-27-114/+148
| | | | | | | | | | | | | | | | | | | | | | | | Two functions were created: send_eval and continue_eval, in order to facilitate testing. The corresponding tests are also changed in this commit.
| | | | | * Merge branch 'master' into eval-enhancementsGravatar Matteo Bertucci2020-02-18-379/+350
| | | | | |\
| | | | | * | Use kwargs to set mock attributesGravatar Matteo2020-02-18-25/+11
| | | | | | |
| | | | | * | Assert return value of Snekbox.post_evalGravatar Matteo2020-02-18-1/+3
| | | | | | |