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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| * \|	Fix linting?	mathsman5133	2020-03-31	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Not sure why my precommit didn't pick that up...
\| \| \| \| * \|	Merge branch 'master' of https://github.com/python-discord/bot into ↵	mathsman5133	2020-03-31	-375/+1918
\| \| \| \| \|\ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	help-refactor
\| \| \| \| * \| \|	Apply suggestions from review.	mathsman5133	2020-03-31	-12/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Make exception handling for bin reaction more specific - Channel constants were updated recently - Suggest category names - Tidy up signature formatting - Move score cutoff to 80 to allow a few more matches
\| \| \| \| * \| \|	Merge branch 'master' of https://github.com/python-discord/bot into ↵	mathsman5133	2020-03-12	-2207/+4239
\| \| \| \| \|\ \ \ \| \| \| \|_\|/ / / \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	help-refactor Conflicts: bot/cogs/help.py
\| \| \| \| * \| \|	Apply suggestions from Mark's code review.	mathsman5133	2020-03-10	-60/+63
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	Use the new :trashcan: emoji to delete the help message, as per #625	mathsman5133	2020-02-10	-4/+5
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	Merge branch 'master' of https://github.com/python-discord/bot into ↵	mathsman5133	2020-02-10	-3174/+7258
\| \| \| \| \|\ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	help-refactor Conflicts: bot/cogs/help.py
\| \| \| \| * \| \| \|	Remove trailing comma	mathsman5133	2019-11-16	-1/+1
\| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \|	Apply suggestions from review	mathsman5133	2019-11-16	-24/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Description was the same as prefix parameter of paginator - Cleanup is redundant pending closure of #514 - Clean/fix couple if statements in help.py
\| \| \| \| * \| \| \|	Show a maximum of 8 commands per page rather than 5.	mathsman5133	2019-11-03	-1/+1
\| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \|	Few changes to keep formatting same as current	mathsman5133	2019-10-13	-5/+16
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Change `add_field` back to `description` for error message possible matches - Only add `Commands` and `Subcommands` if subcommands exist to cog/group/command help
\| \| \| \| * \| \| \|	Add a special case for when the help command invokes wolfram checks.	mathsman5133	2019-10-13	-0/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Before, running `!help` would invoke the cooldown check, and increase the cooldown counter unnecessarily as no wolfram API calls were being made. - Once `!help` was called enough, the bot would send an error embed to let you know your wolfram cooldown has expired.
\| \| \| \| * \| \| \|	Minor formatting changes to align with current help.	mathsman5133	2019-10-12	-2/+2
\| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \|	Refactor the `Help` command.	mathsman5133	2019-10-12	-475/+287
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- `redirect_output` has been adjusted to run the `delete_invocation` inside a task as the help command will wait for that to run before sending the help or doing anything else. - `pagination` has been adjusted to support deleting the paginated message if `cleanup` is True, and an optional `description` that is present through all pages of pagination. - The help command has been refactored to subclass `commands.HelpCommand`. This means that it now supports methods such as `ctx.send_help(ctx.command)`. - `help_cleanup` provides the opportunity to use the :x: reaction to cleanup help even with no pagination. - Pagination purely happens through the `LinePaginator`, forcing a pagination session with 1 line per page where we format the page style before sending it through. - Categories are properly dealt with by finding a match and sending a seperate help where a named tuple of the Category name, description and relevant cogs is the only parameter. - Choices for when a command was not found has been updated to include category names, cog names, aliases of group and command names, and include all subcommands and aliases. This should provide a more helpful output when an error message is sent - Sending command, group, cog, category and bot help has been split into different functions that are called from `command_callback`. This provides an easier way to alter future changes, and cleans up code considerably. - Important note: no outward facing formatting should have changed. Any desired changes can be discussed in review.
\| \| \| \| \| \| \| *	Fix incomplete variable renaming	decorator-factory	2020-05-17	-1/+1
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Rename `string` to `greeting`	decorator-factory	2020-05-17	-10/+10
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Add a note on user-defined classes	decorator-factory	2020-05-17	-1/+1
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Change standalone programs to interactive sessions	decorator-factory	2020-05-17	-8/+13
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Apply language improvements proposed from kwzrd	decorator-factory	2020-05-17	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: kwzrd <[email protected]>
\| \| \| \| \| \| \| *	Fix hard-wrapping in mutability.md	decorator-factory	2020-04-01	-7/+5
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	header->bold in mutability.md	decorator-factory	2020-04-01	-1/+1
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Add mutability.md tag	decorator-factory	2020-04-01	-0/+34
\| \|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \|
* \| \| \| \| \| \|	Merge pull request #852 from ks129/infraction-edit	S. Co1	2020-03-31	-1/+6
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Don't let change expiration time when infraction already expired.
\| * \ \ \ \ \ \	Merge branch 'master' into infraction-edit	S. Co1	2020-03-31	-60/+562
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge pull request #854 from ks129/tags-trashcan-react	S. Co1	2020-03-31	-5/+17
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Tags response + command message deletion with trashcan reaction
\| * \ \ \ \ \ \ \	Merge branch 'master' into tags-trashcan-react	S. Co1	2020-03-31	-55/+545
\| \|\ \ \ \ \ \ \ \ \| \|/ / / / / / / / \|/\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #846 from python-discord/feat/filter/322/merge-tokens-words	S. Co1	2020-03-31	-41/+21
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Filtering: merge the word and token watch filters
\| * \ \ \ \ \ \ \ \	Merge branch 'master' into feat/filter/322/merge-tokens-words	S. Co1	2020-03-31	-15/+617
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #833 from ks129/zen-match-fix	Sebastiaan Zeeff	2020-03-31	-1/+11
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixed !zen command exact word matching.
\| * \ \ \ \ \ \ \ \ \	Merge branch 'master' into zen-match-fix	Karlis S	2020-03-31	-14/+606
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #845 from python-discord/update-logging-levels	Leon Sandøy	2020-03-31	-10/+9
\|\ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \|	Logging Level Adjustments
\| * \| \| \| \| \| \| \| \| \|	Merge branch 'master' into update-logging-levels	Leon Sandøy	2020-03-31	-12/+656
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Set unsilence permissions to inherit instead of true	Sebastiaan Zeeff	2020-03-30	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The "unsilence" action of the silence/hush command used `send_messages=True` when unsilencing a hushed channel. This had the side effect of also enabling send messages permissions for those with the Muted rule, as an explicit True permission apparently overwrites an explicit False permission, even if the latter was set for a higher top-role. The solution is to revert back to the `Inherit` permission by assigning `None`. This is what we normally use when Developers are allowed to send messages to a channel.
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #812 from Numerlor/hush-cog	Eivind Teig	2020-03-30	-3/+504
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Implement silencer (hush) cog.
\| * \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into hush-cog	Eivind Teig	2020-03-30	-548/+1802
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Add tests for `HushDurationConverter`.	Numerlor	2020-03-17	-0/+30
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Add tests ensuring permissions get preserved.	Numerlor	2020-03-17	-0/+31
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Use set `discard` instead of `remove`.	Numerlor	2020-03-17	-3/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Discard ignores non present values, allowing us to skip the KeyError suppress.
\| * \| \| \| \| \| \| \| \| \| \|	Add docstring to test.	Numerlor	2020-03-17	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Fix test name.	Numerlor	2020-03-17	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`removed` was describing the opposite behaviour.
\| * \| \| \| \| \| \| \| \| \| \|	Remove `alert_channel` mention from docstring.	Numerlor	2020-03-17	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	After removing the optional channel arg and changing output message channels we're only testing `ctx`'s `send`.
\| * \| \| \| \| \| \| \| \| \| \|	Remove one indentation level.	Numerlor	2020-03-17	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \|	Reword `unsilence` docstring.	Numerlor	2020-03-17	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \|	Remove `channel` mentions from command docstrings.	Numerlor	2020-03-17	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	With the new behaviour of not accepting channels and muting the current one, it's no longer neccessary to keep the channel param in the docstring. Co-authored-by: MarkKoz <[email protected]>
\| * \| \| \| \| \| \| \| \| \| \|	Rename `cog_unload` tests.	Numerlor	2020-03-17	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previous names were undescriptive from testing phases.
\| * \| \| \| \| \| \| \| \| \| \|	Fix `test_cog_unload` passing tests with invalid values.	Numerlor	2020-03-17	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The first assert - `asyncio_mock.create_task.assert_called_once_with` called `alert_channel`'s send resulting in an extra call. `send` on `alert_channel` was not tested properly because of a typo and a missing assert in the method call.
\| * \| \| \| \| \| \| \| \| \| \|	Handle and log `CommandErrors` on `.can_run`.	Numerlor	2020-03-16	-1/+6
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Remove long indentation from docstrings.	Numerlor	2020-03-15	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Set `_get_instance_vars_event` in test's `setUp`.	Numerlor	2020-03-15	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Adjust tests for new calling behaviour.	Numerlor	2020-03-15	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`.set_permissions` calls were changed to use kwargs directly instead of an overwrite, this reflects the changes in tests.