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

	Commit message (Collapse)	Author	Age	Lines
...
\| * \| \| \| \| \|	Snekbox tests: use `get_code` in `test_continue_eval_does_continue`	MarkKoz	2020-03-25	-3/+6
\| \| \| \| \| \| \|
\| * \| \| \| \| \|	Snekbox tests: test `get_code`	MarkKoz	2020-03-25	-4/+32
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Should return 1st arg (or None) if eval cmd in message, otherwise return full content.
\| * \| \| \| \| \|	Snekbox: fix re-eval when '!eval' is removed from edited message	MarkKoz	2020-03-25	-1/+21
\|/ / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The previous parsing method was naïve in assuming there would always be something preceding the code (e.g. the '!eval' command invocation) delimited by a space. Now it will only split if it's sure the eval command was used in the edited message.
* \| \| \| \| \|	Merge pull request #840 from ks129/zen-tag-remove	Mark	2020-03-25	-20/+0
\|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Removed `zen` tag due `!zen` command exist.
\| * \ \ \ \ \	Merge branch 'master' into zen-tag-remove	Mark	2020-03-25	-254/+259
\| \|\ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|
* \| \| \| \| \| \|	Merge pull request #841 from python-discord/update-deps	scragly	2020-03-25	-179/+98
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove unused safety & dodgy dev dependencies
\| * \| \| \| \| \| \|	Remove unused safety & dodgy dev dependencies	S. Co1	2020-03-24	-179/+98
\|/ / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Relock
* \| \| \| \| \| \|	Merge pull request #837 from python-discord/update-contrib	kwzrd	2020-03-23	-22/+37
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Update contributor doc
\| * \ \ \ \ \ \	Merge branch 'master' into update-contrib	kwzrd	2020-03-23	-1/+1
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Fix regression in verification cog	S. Co1	2020-03-21	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A stray `bot` was removed from the `on_message` listener, causing it to raise an exception rather than generate a `Context` object from incoming verification channel messages.
\| * \| \| \| \| \| \|	Update contributor doc	S. Co1	2020-03-20	-22/+37
\|/ / / / / / /
* \| \| \| \| \| \|	Merge pull request #821 from python-discord/hemlock-perma-ban-watch-removal	S. Co1	2020-03-20	-44/+94
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Automatically Remove Users from BigBrother Watch List on Perma Ban
\| * \| \| \| \| \| \|	Changed a logging level	Daniel Brown	2020-03-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Changed the log for when the big brother cog doesn't load in the `apply_ban()` method doesn't properly load from a trace to an error.
\| * \| \| \| \| \| \|	Merge branch 'master' into hemlock-perma-ban-watch-removal	Daniel Brown	2020-03-20	-117/+1052
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge pull request #530 from python-discord/spoiler-check	Leon Sandøy	2020-03-17	-10/+31
\|\ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|/ / / \|/\| \| \| \| \| \| \|	Strip spoiler tags for watchlist triggers
\| * \| \| \| \| \| \|	Merge branch 'master' into spoiler-check	Jeremiah Boby	2020-03-17	-107/+1021
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Escape markdown in voice updates	Jeremiah Boby	2020-03-11	-7/+14
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Escape markdown in member updates	Jeremiah Boby	2020-03-11	-1/+3
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Escape markdown in watchlist triggers	Jeremiah Boby	2020-03-11	-2/+3
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Merge branch 'master' into spoiler-check	Jeremiah Boby	2020-03-11	-965/+903
\| \|\ \ \ \ \ \ \
\| * \ \ \ \ \ \ \	Merge branch 'master' into spoiler-check	Leon Sandøy	2020-02-29	-297/+857
\| \|\ \ \ \ \ \ \ \
\| * \ \ \ \ \ \ \ \	Merge branch 'master' into spoiler-check	Jeremiah Boby	2020-02-27	-2042/+5374
\| \|\ \ \ \ \ \ \ \ \
\| * \ \ \ \ \ \ \ \ \	Merge branch 'master' into spoiler-check	Jeremiah Boby	2019-11-28	-2694/+5095
\| \|\ \ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \| \| \|	Expand spoilers to match multiple interpretations	Jeremiah Boby	2019-11-28	-1/+11
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Update spoiler regex to support multi-line spoilers	Jeremiah Boby	2019-10-21	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Check partially hidden words against the wordlist	Jeremiah Boby	2019-10-13	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \|	Updated Docstrings, parameters, and log messages	Daniel Brown	2020-03-20	-13/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Docstrings for `apply_ban()` have been edited to mention that the method also removes a banned user from the watch list. - Parameter `banned` in `apply_unwatch()` was changed to `send_message` in order to be more general. Boolean logic was swapped to coincide with that change. - `apply_unwatch()`'s sent message moved to the bottom of the method for clarity. Added `return`s to the method to exit early if no message needs to be sent. Signed-off-by: Daniel Brown <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \|	Updated doc strings to be more descriptive	Daniel Brown	2020-03-20	-2/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Mark <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \|	Nesting reduced, logging cleaned up and made clearer	Daniel Brown	2020-03-16	-12/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Mark <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \|	Update bot/cogs/moderation/infractions.py	Daniel Brown	2020-03-14	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Mark <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \|	Verification Cog Kaizen Changes	Daniel Brown	2020-03-13	-35/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Kaizen: - Cut down on the size of the import line by changing the imports from bot.constants to instead just importing the constants. This will help clarify where certain constants are coming from. - The periodic checkpoint message will no longer ping `@everyone` or `@Admins` when the bot detects that it is being ran in a debug environment. Message is now a simple confirmation that the periodic ping method successfully ran. Signed-off-by: Daniel Brown <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \|	Corrected expiration check logic and cog loading	Daniel Brown	2020-03-12	-2/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Bugs fixed: - Previously, the code would check to see if `'expires_at'` was in the kwargs, which after testing I came to find out that it is regardless of the duration of the ban. It has sense been changed to use a `.get()` in order to do a proper comparison. - Code previously attempted to load from the `"BigBrother"` cog which is the incorrect spelling. Changed it to `"Big Brother"` to correct this. Logging Added: - Additional trace logs added to both the `infractions.py` file as well as `bigbrother.py` to assist with future debugging or testing. Signed-off-by: Daniel Brown <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \|	Merge remote-tracking branch 'origin/hemlock-perma-ban-watch-removal' into ↵	Daniel Brown	2020-03-10	-0/+0
\| \| \| \| \| \| \|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	hemlock-perma-ban-watch-removal
\| \| \| \| \| \| \| * \| \| \| \| \|	Added BigBrother Helper Methods	Daniel Brown	2020-03-06	-7/+28
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Added apply_unwatch() and migrated the code from the unwatch command to it. This will give us more control regarding testing and also determining when unwatches trigger. - Added apply_watch() and migrated the code from the watch command to it. Again, this will assist with testing and could make it easier to automate adding to the watch list if need be. - Added unwatch call to apply_ban. User will only be removed from the watch list if they were permanently banned. They will not be removed if it was only temporary. Signed-off-by: Daniel Brown <[email protected]>
\| \| \| \| \| \| * \| \| \| \| \| \|	Added BigBrother Helper Methods	Daniel Brown	2020-03-09	-7/+28
\| \| \| \| \| \|/ / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Added apply_unwatch() and migrated the code from the unwatch command to it. This will give us more control regarding testing and also determining when unwatches trigger. - Added apply_watch() and migrated the code from the watch command to it. Again, this will assist with testing and could make it easier to automate adding to the watch list if need be. - Added unwatch call to apply_ban. User will only be removed from the watch list if they were permanently banned. They will not be removed if it was only temporary. Signed-off-by: Daniel Brown <[email protected]>
\| \| \| \| \| \| \| * / / / /	Removed `zen` tag due `!zen` command exist.	Karlis S	2020-03-23	-20/+0
\| \|_\|_\|_\|_\|_\|/ / / / / \|/\| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| * \|	Clarify docstring for token check function	S. Co1	2020-03-23	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Mark <[email protected]>
\| \| \| \| \| \| \| \| \| * \|	Update token filter logging to match expanded detection	S. Co1	2020-03-20	-12/+18
\| \|_\|_\|_\|_\|_\|_\|_\|/ / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Log message still used the first regex result (re.search) rather than the expanded approach (re.findall) recently added.
* \| \| \| \| \| \| \| \| \|	Merge pull request #830 from ks129/bug-fixes	Mark	2020-03-16	-1/+4
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	ModLog Channel Update formatting fix.
\| * \ \ \ \ \ \ \ \ \	Merge branch 'master' into bug-fixes	Mark	2020-03-16	-0/+80
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #828 from python-discord/feat/test/765/command-shadowing	Mark	2020-03-16	-0/+80
\|\ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|/ / \|/\| \| \| \| \| \| \| \| \| \|	Test for command name and alias shadowing
\| * \| \| \| \| \| \| \| \| \|	Merge branch 'master' into feat/test/765/command-shadowing	Mark	2020-03-16	-106/+937
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Cog tests: comment some code for clarification	MarkKoz	2020-03-16	-0/+3
\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Cog tests: fix error on import due to discord.ext.tasks.loop	MarkKoz	2020-03-13	-3/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The tasks extensions loop requires an event loop to exist. To work around this, it's been mocked.
\| * \| \| \| \| \| \| \| \| \|	Cog tests: fix leading space in aliases without parents	MarkKoz	2020-03-13	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Cog tests: add a test for duplicate command names & aliases	MarkKoz	2020-03-13	-0/+17
\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Cog tests: fix duplicate commands being yielded	MarkKoz	2020-03-13	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	discord.py yields duplicate Command objects for each alias a command has, so the duplicates need to be removed on our end.
\| * \| \| \| \| \| \| \| \| \|	Cog tests: fix nested modules not being found	MarkKoz	2020-03-13	-9/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Rename `walk_extensions` to `walk_modules` because some extensions don't consist of a single module
\| * \| \| \| \| \| \| \| \| \|	Cog tests: fix duplicate cogs being yielded	MarkKoz	2020-03-13	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Have to check the modules are equal to prevent yielding imported cogs.
\| * \| \| \| \| \| \| \| \| \|	Cog tests: fix Cog type check in `walk_cogs`	MarkKoz	2020-03-13	-3/+3
\| \| \| \| \| \| \| \| \| \| \|