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

	Commit message (Collapse)	Author	Lines
2020-03-01	!roles Command: Added pagination (LinePaginator), moved roles amount to ↵	Karlis. S	-8/+7
	title (was before in footer).
2020-02-28	Scheduler: correct type annotations	MarkKoz	-6/+6

2020-02-28	Snekbox: mention re-evaluation feature in the command's docstring	MarkKoz	-1/+4

2020-02-28	Ignore NotFound errors inside continue_eval	Matteo	-24/+25
	It could have caused some errors if the user delete his own message
2020-02-28	Return only the new code in continue_eval and check for truthiness instead	Matteo	-11/+10

2020-02-27	Scheduler: log the exception instead of raising	MarkKoz	-2/+5
	Logging it ourselves has a cleaner traceback and gives more control over the output, such as including the task ID.
2020-02-27	Remove unnecessary newlines from call.	Numerlor	-4/+1

2020-02-27	Moderation: fix task cancellation for permanent infraction when editing	MarkKoz	-1/+3
	A task should not be cancelled if an infraction is permanent because tasks don't exist for permanent infractions. Fixes BOT-1V
2020-02-27	Use unicode code point instead of literal for the snekbox re-eval emoji	Matteo	-6/+8
	Unicode literals aren't really safe compared to code points
2020-02-27	Split the eval command procedure into two functions.	Matteo	-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.
2020-02-26	Add clarifying comment to role checking logic implementation	S. Co1	-0/+1

2020-02-26	Log invalid tag names in the error handler tag fallback.	Numerlor	-5/+13

2020-02-26	Remove logging from tag converters.	Numerlor	-10/+0

2020-02-26	Disable TRACE logging for Sentry breadcrumbs.	Joseph	-1/+1

2020-02-26	Adjust tests for new converter behavior.	Numerlor	-1/+1

2020-02-26	Remove number check on tags.	Numerlor	-14/+0
	This case is already covered by checking if at least one letter is included.
2020-02-26	Pass error handler tag fallback through TagNameConverter.	Numerlor	-2/+6
	The tag fallback didn't convert tags, resulting in possible invalid tag names being passed to the `tags_get_command`. This makes sure they're valid and ignores the risen exception if they are not.
2020-02-26	Make sure tag name contains at least one letter.	Numerlor	-0/+6
	With only ascii and numbers being allowed to go through, possible values still included things like `$()` which don't match anything in `REGEX_NON_ALPHABET` from tags.py resulting in an error.
2020-02-26	Added DMs ignoring to antimalware check	ks123	-1/+2

2020-02-26	Fix comparison operator when checking token expiration.	Numerlor	-1/+1
	With `<` the check only went through when the token was already expired, making revoking redundant; and didn't go through when the token still had some time before expiration.
2020-02-26	Added to AntiMalware staff ignore check.	Karlis. S	-1/+5

2020-02-26	Check for falsy values instead of ``""` and `None` explicitly.	Numerlor	-2/+1
	After the change to also check empty strings to avoid unucessary requests, it is no longer necessary to do an explicit value check, as the only values that can come from the .env file are `None` and strings Co-authored-by: Karlis S <[email protected]>
2020-02-26	Specify the logged time is in UTC.	Numerlor	-1/+1

2020-02-26	Create task for `revoke_access_token` when unloading cog to ensure it's ↵	Numerlor	-1/+1
	executed.
2020-02-25	Scheduler: only send warning in callback if task isn't cancelled	MarkKoz	-1/+1
	If a task is cancelled it is assumed it was done via cancel_task. That method deletes the task after cancelling so the warning isn't relevant.
2020-02-25	Scheduler: properly raise task's exception the done callback	MarkKoz	-1/+4
	Task.exception() only returns the exception. It still needs to be explicitly raised.
2020-02-25	Scheduler: only delete the task in the done callback if tasks are same	MarkKoz	-16/+25
	To prevent a deletion of task rescheduled with the same ID, the callback checks that the stored task is the same as the done task being handled. * Only delete the task; it doesn't need to be cancelled because the it is already done * Revise the callback's docstring to explain the new behaviour * Rename `task` parameter to `done_task`
2020-02-25	Scheduler: improve cancel_task's docstring	MarkKoz	-5/+5
	* Use imperative mood for docstring * Explain the purpose of the parameter in the docstring * Make log message after cog name lowercase
2020-02-25	Moderation: use asyncio.shield to prevent self-cancellation	MarkKoz	-3/+4
	The shield exists to be used for exactly this purpose so its a better fit than create_task.
2020-02-26	Make sure token exists before checking its expiration.	Numerlor	-1/+1
	Without the check and an invalid token, an AttributeError is raised; blocking the relevant ClientError from being raised in `get_access_token`.
2020-02-26	Check for empty strings alongside None before loading cog.	Numerlor	-1/+2
	Docker fetches values from the .env itself and defaults to "" instead of None, needing to do invalid access token requests before unloading itself.
2020-02-25	Restyle if body to include the error instead of adding the cog.	Numerlor	-3/+3

2020-02-25	Fix `__iter__` for classes without subsections.	Numerlor	-2/+3
	The previous implementation assumed the config class was a subsection, failing with a KeyError if it wasn't one. Co-authored-by: kwzrd <[email protected]>
2020-02-25	Fix mismatched constant names in syncer tests	S. Co1	-6/+6

2020-02-25	Configure staff role & channel groupings in YAML	S. Co1	-10/+27
	Delete duplicate keys that were missed in the merge
2020-02-25	Assign created task to a variable.	Numerlor	-1/+1
	Co-authored-by: SebastiaanZ <[email protected]>
2020-02-25	Wait for available guild instead of bot startup.	Numerlor	-1/+1
	Co-authored-by: SebastiaanZ <[email protected]>
2020-02-24	Do not attempt to load Reddit cog when environment variables are not provided.	Numerlor	-1/+4
	When environment variables weren't provided; the cog attempted to create a BasicAuth object with None as values resulting in an exception before the event loop was started and a subsequent crash.
2020-02-24	Remove unused `chunks` function and its tests.	Numerlor	-26/+1
	The function was only used in the since removed `Events` cog.
2020-02-24	Change docstring mood.	Numerlor	-1/+1

2020-02-24	Change warning text.	Numerlor	-1/+1

2020-02-24	Add ConfigVerifier cog.	Numerlor	-0/+41
	Adds ConfigVerifier which verifies channels when loaded.
2020-02-24	Implement `__iter__` on constants YAMLGetter.	Numerlor	-0/+4
	Python tries to fall back on passing indices to `__getitem__` without iter implemented; failing on the first line.
2020-02-23	Sync: code style refactoring	MarkKoz	-7/+10
	* Convert diff namedtuple to dict outside the dict comprehension * Define long condition as a boolean instead of in the if statement * Pass role and user dicts to aiohttp normally instead of unpacking
2020-02-23	Tests: fix setting bot-commands ID in information tests	MarkKoz	-4/+4

2020-02-23	Constants: rename Guild.Constant.ignored to modlog_blacklist	MarkKoz	-7/+8
	This name better explains what the list is for.
2020-02-23	Config: replace abbreviated lists with normal ones	MarkKoz	-4/+34
	Lists were getting too long to be readable as one line. Having each element on a separate line also reduces merge conflicts.
2020-02-23	Config: add underscore to devcontrib	MarkKoz	-2/+2

2020-02-23	Config: fix DEV_LOG variable thingy	MarkKoz	-2/+2

2020-02-23	Always load doc and verification extensions	MarkKoz	-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.