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

	Commit message (Collapse)	Author	Age	Lines
...
* \| \| \| \| \| \|	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
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Cog tests: add a function to yield all commands	MarkKoz	2020-03-13	-0/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This will help reduce nesting in the actual test.
\| * \| \| \| \| \| \| \|	Cog tests: add a function to get all qualified names for a cmd	MarkKoz	2020-03-13	-0/+8
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Cog tests: add a function to get all cogs	MarkKoz	2020-03-13	-0/+7
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Cog tests: add a function to get all extensions	MarkKoz	2020-03-13	-0/+11
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Cog tests: add a function to get all commands	MarkKoz	2020-03-09	-0/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	For tests, ideally creating instances of cogs should be avoided to avoid extra code execution. This function was copied over from discord.py because their function is not a static method, though it still works as one. It was probably just a design decision on their part to not make it static.
\| * \| \| \| \| \| \| \|	Cog tests: create boilerplate for command name tests	MarkKoz	2020-03-09	-0/+7
\| \| \|_\|_\|_\|_\|/ / \| \|/\| \| \| \| \| \|
\| \| * \| \| \| \| \|	Update explanation comment so it explains what happens	S. Co1	2020-03-16	-2/+3
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	(Mod Log): Added comment about channel update formatting change.	Karlis S	2020-03-16	-0/+2
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	(Mod Log): Fixed case when `on_guild_channel_update` old or new value is ↵	ks123	2020-03-16	-1/+1
\| \|/ / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	empty and with this message formatting go wrong.
* \| \| \| \| \| \|	Merge pull request #803 from RohanJnr/tags_overhaul	Shirayuki Nekomata	2020-03-16	-106/+937
\|\ \ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|	Removed API dependency for Tags
\| * \| \| \| \| \|	Fixed tag search via contents, any keywords.	Shirayuki Nekomata	2020-03-16	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixed `!tag search any` raises `AttributeError`. Changed default value of `keywords` from `None` to `'any'`. This will make it search for keyword `'any'` when there is no keyword.
\| * \| \| \| \| \|	not awaiting _get_tags_via_content() method as it is non-async	RohanJnr	2020-03-15	-2/+2
\| \| \| \| \| \| \|
\| * \| \| \| \| \|	Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵	RohanJnr	2020-03-15	-9/+15
\| \|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tags_overhaul
\| \| * \| \| \| \| \|	Tags: add restrictions 1 & 9 from YouTube ToS to ytdl tag	MarkKoz	2020-03-14	-2/+6
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	Tags: use constant for command prefix in embed footer	MarkKoz	2020-03-14	-7/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Add a constant for the footer text * Import constants module rather than its classes
\| * \| \| \| \| \| \|	convert _get_tags_via_content() method to non-async	RohanJnr	2020-03-15	-1/+1
\| \|/ / / / / /
\| * \| \| \| \| \|	Remove line that calls get_tags() method	RohanJnr	2020-03-14	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The tags have now been shifted from the database to being static files and hence the get_tags() method has undergone changes. It now dosen't fetch from the database but looks at the local files and we need not call it more than once.
\| * \| \| \| \| \|	Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵	RohanJnr	2020-03-12	-22/+200
\| \|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tags_overhaul
\| \| * \ \ \ \ \	Merge branch 'master' into tags_overhaul	Rohan	2020-03-12	-22/+200
\| \| \|\ \ \ \ \ \ \| \|_\|/ / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge pull request #822 from python-discord/bug/mod/792/null-attachments	Eivind Teig	2020-03-09	-1/+1
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	ModLog: fix posting null attachments for deleted message logs
\| * \ \ \ \ \ \ \	Merge branch 'master' into bug/mod/792/null-attachments	Eivind Teig	2020-03-09	-6/+103
\| \|\ \ \ \ \ \ \ \ \| \|/ / / / / / / / \|/\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #825 from python-discord/token-detection-fix	Eivind Teig	2020-03-09	-3/+10
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Refactor token detection to check all potential substrings in message
\| * \ \ \ \ \ \ \ \	Merge branch 'master' into token-detection-fix	Eivind Teig	2020-03-09	-3/+93
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #826 from python-discord/vote	Eivind Teig	2020-03-09	-0/+21
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Implement vote command.
\| * \| \| \| \| \| \| \| \| \|	Disambiguate codepoint value.	scragly	2020-03-09	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The usage of 127462 as a unicode start point isn't super clear for other devs coming across the code in future, so assigning it to a nicely named variable with an accompanying inline comment should help make things clearer.
\| * \| \| \| \| \| \| \| \| \|	Check lower bound for vote options.	scragly	2020-03-09	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If the vote command receives less than 2 options, it's not being used for it's intended usage and is considered a user input error.
\| * \| \| \| \| \| \| \| \| \|	Merge branch 'master' into vote	scragly	2020-03-09	-2/+2
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #824 from python-discord/antimalware-fix	Mark	2020-03-08	-2/+2
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix filtered extension string out of scope for antismalware log message
\| * \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into antimalware-fix	Mark	2020-03-08	-1/+70
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Fix filtered extension string out of scope for log message	S. Co1	2020-03-08	-2/+2
\| \| \|_\|_\|_\|_\|_\|_\|_\|_\|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Fix typo in file extensions list comprehension
\| \| * \| \| \| \| \| \| \| \|	Implement vote command.	scragly	2020-03-09	-0/+18
\| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The vote command takes a given list of options and generates a simple message and corresponding reactions for each so members can quickly take a vote on a subject during in-server discussions and meetings.
* \| \| \| \| \| \| \| \| \|	Merge pull request #823 from ↵	Mark	2020-03-08	-1/+70
\|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	python-discord/tag-search-searches-tags-via-content-instead-of-names Subcommand `search` and `search any` to search inside tags' contents and not names.
\| * \| \| \| \| \| \| \| \|	Refactored to use paginator like normal `!tag`	Shirayuki Nekomata	2020-03-09	-12/+20
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Split `_get_tags_via_content` - introduce `_send_matching_tags` - `_send_matching_tags` will send and paginate like `!tag` - Simplified `is_plural` even more.
\| * \| \| \| \| \| \| \| \|	Fixed `is_plural` counting 1 less space.	Shirayuki Nekomata	2020-03-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-Authored-By: Mark <[email protected]>
\| * \| \| \| \| \| \| \| \|	Refactored dense codes, removed obvious type hint.	Shirayuki Nekomata	2020-03-08	-15/+25
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Show the process of sanitizing the List[str] `keywords_processed`. - Show the process of finding tag for `matching_tags` ( was `founds` ). - Refactored the logic to find boolean `is_plural`. - Minor wording changes for docstring.
\| * \| \| \| \| \| \| \| \|	Fixed searching for `,` returing all tags. Made it more descriptive when ↵	Shirayuki Nekomata	2020-03-08	-3/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	multiple tags are found. - Added a truthy check for each `query` since `','.split()` returns a list of two empty strings. - Changed from `Did you mean ...` to `Here are the tags containing the given keyword(s):` to be much more descriptive about the results - they are `tag` and not `term` to be searched.
\| * \| \| \| \| \| \| \| \|	Refactored tag searching via keywords in contents	Shirayuki Nekomata	2020-03-08	-8/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Refactored `if` block - change to only send result when there is any result. - Added better type hinting for `check` argument of `_get_tags_via_content` - changed from `callable` to `Callable[[Iterable], bool]`. Thanks to @markkoz 's reviews Co-Authored-By: Mark <[email protected]>
\| * \| \| \| \| \| \| \| \|	Implemented `search` as a subcommand for `tag` that will search in contents ↵	Shirayuki Nekomata	2020-03-08	-1/+50
\|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	instead of names - `!tag search` will search for multiple keywords, separated by comma, and return tags that has ALL of these keywords. ` !tag search any` is the same as `!tag search` but it return tags that has ANY of the keyword instead.