bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Age	Lines
...
\| * \| \| \| \| \| \| \| \| \| \| \|	Use kwargs to set mock attributes	Matteo	2020-02-18	-25/+11
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Assert return value of Snekbox.post_eval	Matteo	2020-02-18	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Use a space instead of an empty string in test_get_status_emoji	Matteo Bertucci	2020-02-18	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Because of the stripping, it should still be considered as empty Co-Authored-By: Mark <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Delete additional informations from subtest	Matteo Bertucci	2020-02-18	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Reduce visual clutter Co-Authored-By: Mark <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Split assertions onto separate lines	Matteo Bertucci	2020-02-18	-4/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Reads better as separate lines Co-Authored-By: Mark <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Split assertion onto separate lines	Matteo Bertucci	2020-02-18	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Mark <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Snekbox small refactoring	Matteo	2020-02-18	-4/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Makes the code a bit clearer Co-authored-by: Shirayuki Nekomata <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \| \|	Suppress HTTPException while deleting bot output	Matteo	2020-02-18	-2/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It was triggering an error if the user deleted the output before re-evaluating
\| * \| \| \| \| \| \| \| \| \| \| \|	Use pregenerated partials	Matteo	2020-02-18	-2/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This avoid recreating partials for each re-eval
\| * \| \| \| \| \| \| \| \| \| \| \|	Write tests for bot/cogs/test_snekbox.py	Akarys42	2020-02-09	-0/+363
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Create an AsyncContextManagerMock mock for testing asynchronous context managers	Akarys42	2020-02-09	-0/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It can be used to test aiohttp request functions, since they are async context managers
\| * \| \| \| \| \| \| \| \| \| \| \|	Use actual functions instead of lambdas for bot.wait_for	Akarys42	2020-02-09	-2/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The use of lambdas made the functions hard to test, this new format allows us to easily test those functions and document them.
\| * \| \| \| \| \| \| \| \| \| \| \|	Merge branch 'master' into eval-enhancements	Matteo Bertucci	2020-01-30	-350/+648
\| \|\ \ \ \ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Re-eval snippet with emoji reaction	Matteo Bertucci	2020-01-29	-23/+46
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If the eval message is edited after less than 10 seconds, an emoji is added to the message, and if the user adds the same, the snippet is re-evaluated. This make easier to correct snipper mistakes.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Don't strip whitespaces during snekbox formatting	Matteo Bertucci	2020-01-28	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It could lead to a misleading result if it is stripped.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Correct eval output to include the 11th line	Akarys42	2019-12-26	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Hardcode SIGKILL value	Akarys42	2019-12-25	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It allows the cog to also work on Windows, because of Signals.SIGKILL not being defined on this platform
\| \| \| \| * \| \| \| \| \| \| \| \| \|	Merge branch 'master' into fuzzy_zero_div	Numerlor	2020-02-28	-273/+386
\| \| \| \| \|\ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	# Conflicts: # bot/cogs/error_handler.py
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #757 from python-discord/feat/backend/b131/error-handling	Leon Sandøy	2020-02-28	-102/+149
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Handle more types of CommandError
\| * \ \ \ \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into feat/backend/b131/error-handling	Leon Sandøy	2020-02-28	-196/+294
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #782 from python-discord/feat/backend/b496/rename-constants	Leon Sandøy	2020-02-28	-163/+229
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|	Rename constants to match their guild counterparts
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge branch 'master' into feat/backend/b496/rename-constants	Leon Sandøy	2020-02-28	-33/+65
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix mismatched constant names in syncer tests	S. Co1	2020-02-25	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Configure staff role & channel groupings in YAML	S. Co1	2020-02-25	-10/+27
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Delete duplicate keys that were missed in the merge
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge branch 'master' into feat/backend/b496/rename-constants	S. Co1	2020-02-25	-546/+1676
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Tests: fix setting bot-commands ID in information tests	MarkKoz	2020-02-23	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Constants: rename Guild.Constant.ignored to modlog_blacklist	MarkKoz	2020-02-23	-7/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This name better explains what the list is for.
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: replace abbreviated lists with normal ones	MarkKoz	2020-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 devcontrib	MarkKoz	2020-02-23	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: fix DEV_LOG variable thingy	MarkKoz	2020-02-23	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Always load doc and verification extensions	MarkKoz	2020-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 guild	MarkKoz	2020-02-23	-76/+76
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Constants: remove code jam champions role	MarkKoz	2020-02-23	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Nothing was using it.
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Constants: rename developers role back to verified	MarkKoz	2020-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 guild	MarkKoz	2020-02-23	-28/+28
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: remove checkpoint_test and devtest	MarkKoz	2020-02-23	-6/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	They no longer exist in the guild. * Move devlog under the "Logs" category
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: shorten name of PYTHON_COMMUNITY_ROLE	MarkKoz	2020-02-23	-7/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: split channels into categories	MarkKoz	2020-02-23	-21/+39
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: split roles into categories	MarkKoz	2020-02-23	-12/+16
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Config: rename roles to match their names in the guild	MarkKoz	2020-02-23	-18/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Merge remote-tracking branch 'origin/master' into ↵	MarkKoz	2020-02-25	-1158/+2657
\| \| \| \|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \|/ / / / / / / / / / / / / \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	feat/backend/b131/error-handling
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Error handler: handle CommandOnCooldown errors	MarkKoz	2020-02-16	-4/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply send the error's default message to the invoking context.
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Error handler: update docstrings to reflect recent changes	MarkKoz	2020-02-16	-20/+39
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Error handler: simplify check failure handler & handle bot missing roles	MarkKoz	2020-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 handled	MarkKoz	2020-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 ArgumentParsingError	MarkKoz	2020-02-16	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply send the error message with the help command.
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Error handler: handle BadUnionArgument	MarkKoz	2020-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 TooManyArguments	MarkKoz	2020-02-16	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Send a message specifying the error reason.
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Error handler: handle MissingRequiredArgument	MarkKoz	2020-02-16	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Send a message indicating which argument is missing.
\| \| \| * \| \| \| \| \| \| \| \| \| \| \| \|	Error handler: simplify error imports	MarkKoz	2020-02-16	-29/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Import the errors module and qualify the error types with it rather than importing a large list of error types.