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

	Commit message (Collapse)	Author	Age	Lines
...
* \| \| \| \| \| \| \| \| \| \| \|	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.
\| * / / / / / / / / /	Refactor token detection to check all potential substrings in message	S. Co1	2020-03-08	-3/+10
\|/ / / / / / / / / /
\| * / / / / / / / /	ModLog: fix posting null attachments for deleted message logs	MarkKoz	2020-03-07	-1/+1
\|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If attachments are not given to `upload_log`, an empty list is used. By default, `zip_longest` uses `None` ass the fill value, so each message was getting paired with a `None` (AKA null) attachment. The filed in the DB is non-nullable so an empty list must be used instead. Fixes #792
* \| \| \| \| \| \| \| \|	Merge pull request #820 from python-discord/feat/ci/pre-commit-tweaks	Eivind Teig	2020-03-06	-2/+1
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Pre-commit tweaks
\| * \| \| \| \| \| \| \| \|	Remove excludes from pre-commit	MarkKoz	2020-03-05	-1/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It was excluding files that are already ignored by git. Pre-commit respects git ignore, even with --all-files, so these ignores were redundant.
\| * \| \| \| \| \| \| \| \|	CI: don't show diff after pre-commit hooks	MarkKoz	2020-03-05	-1/+1
\|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's noisy, messy output. It's not of much benefit anyway as users can run git diff locally if they really need to see a diff. They have to do work locally anyway since CI won't commit the fixes pre-commit makes.
* \| \| \| \| \| \| \| \|	Merge pull request #643 from python-discord/zen-command	scragly	2020-03-05	-2/+86
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	implement !zen command.
\| * \ \ \ \ \ \ \ \	Merge branch 'master' into zen-command	scragly	2020-03-05	-11/+9
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #807 from ks129/roles-command-pagination	scragly	2020-03-05	-11/+9
\|\ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / / \|/\| \| \| \| \| \| \| \| \|	Added Pagination to !roles command
\| * \| \| \| \| \| \| \| \|	Merge branch 'master' into roles-command-pagination	scragly	2020-03-05	-150/+121
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	(Information Cog, !roles command test): Removed 's' at end of "Total 1 ↵	ks123	2020-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	role(s)" due changes in command.
\| * \| \| \| \| \| \| \| \|	(Information Cog, !roles command): Added 's' to end of 'role' only if there ↵	ks123	2020-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	is more then 1 role.
\| * \| \| \| \| \| \| \| \|	(Information Cog, !roles command test): Applied empty parameter change.	ks123	2020-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	(Information Cog, !roles command): Added empty parameter to pagination (False)	ks123	2020-03-05	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Merge branch 'master' into roles-command-pagination	S. Co1	2020-03-03	-735/+507
\| \|\ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \| \|	!roles Command Test: Applied !roles command changes	Karlis. S	2020-03-01	-3/+2
\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	!roles Command: Added pagination (LinePaginator), moved roles amount to ↵	Karlis. S	2020-03-01	-8/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	title (was before in footer).
\| \| \| * \| \| \| \| \| \| \|	Raise BadArgument if no string match.	scragly	2020-03-05	-0/+3
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	Re-use embed, use command converter, raise BadArgument.	scragly	2020-03-05	-37/+22
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	Merge branch 'master' into zen-command	scragly	2020-03-05	-2673/+5311
\| \| \| \|\ \ \ \ \ \ \ \ \| \|_\|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #814 from python-discord/antimalware-logging	Mark	2020-03-04	-17/+24
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Expand Antimalware User Feedback & Logging
\| * \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into antimalware-logging	Mark	2020-03-04	-133/+97
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #816 from ↵	Shirayuki Nekomata	2020-03-04	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	python-discord/extensions-cog-docstring-deprecation-warning Use raw strings for docstrings with forward slashes
\| * \ \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into extensions-cog-docstring-deprecation-warning	Shirayuki Nekomata	2020-03-04	-128/+85
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #811 from python-discord/feat/deps/o138/pre-commit-hooks	scragly	2020-03-04	-128/+85
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add more pre-commit hooks and run them in CI
\| * \ \ \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into feat/deps/o138/pre-commit-hooks	scragly	2020-03-04	-33/+102
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Update code for pep8-naming compliance	S. Co1	2020-03-03	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Remove CaseInsensitiveDict	S. Co1	2020-03-03	-94/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This was added by the now-removed Snake cog & is not used elsewhere on bot.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add pep8-naming & relock	S. Co1	2020-03-03	-14/+28
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	CI: mock the pipenv binary	MarkKoz	2020-03-03	-0/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The mock gets used by the flake8 pre-commit hook, which invokes flake8 via `pipenv run flake8`. It's normally useful to use pipenv here cause it ensures flake8 is invoked within the context of the venv. However, in CI, there is no venv - dependencies are installed directly to the system site-packages. `pipenv run` does not work in such case because it tries to create a new venv if one doesn't exist (it doesn't consider the system interpreter to be a venv). This workaround (okay, it's a hack) creates an executable shell script which replaces the original pipenv binary. The shell script simply ignores the first argument (i.e. ignores `run` in `pipenv run`) and executes the rest of the arguments as a command. It essentially makes `pipenv run flake8` equivalent to just having ran `flake8`. When pre-commit executes pipenv, the aforementioned script is what will run.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	CI: add a restore key for the pre-commit cache	MarkKoz	2020-03-03	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A cache for an outdated pre-commit environment may still be useful. It may be the case that only some hooks need to be updated rather than all.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Use pre-commit in pipenv lint script	MarkKoz	2020-03-03	-4/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add more pre-commit hooks	MarkKoz	2020-03-02	-10/+27
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Remove trailing whitespaces * Specify error code for a noqa in the free command
\| \| * \| \| \| \| \| \| \| \| \| \| \|	Use raw strings for docstrings with forward slashes	Sebastiaan Zeeff	2020-03-04	-3/+3
\| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A few docstrings in `bot.cogs.extensions` have forward slashed in them to escape Markdown rendering when our help feature uses these docstring in a Discord message. However, the use of forward slashes with an invalid escape sequence in docstrings now raises a DeprecationWarning in Python: /home/sebastiaan/pydis/repositories/bot/bot/cogs/extensions.py:72: DeprecationWarning: invalid escape sequence \* PEP 257 (Docstring Conventions, https://www.python.org/dev/peps/pep-0257/) states that raw strings should be used for docstrings that use forward slashes, so I've added the `r`-prefix to the docstrings that use forward slashes.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #815 from python-discord/watchchannel-dm	scragly	2020-03-04	-2/+9
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	Prevent exception if a watched user sends a DM to the bot
\| * \| \| \| \| \| \| \| \| \| \| \|	Prevent exception if a watched user sends a DM to the bot	S. Co1	2020-03-03	-2/+9
\|/ / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The previous embed assumed that the messages would be sent on the server, where the channel would have a name and the message would have a jump URL. For a DM, neither of these are present and an exception will be raised when attempting to construct the embed for the webhook to send.
\| \| * \| \| \| \| \| \| \| \| \|	Merge branch 'master' into antimalware-logging	S. Co1	2020-03-03	-1076/+1451
\| \| \|\ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #750 from python-discord/bug/backend/b748/resolver-in-coro	Mark	2020-03-03	-31/+93
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|	Create AsyncResolver inside a coroutine to avoid DeprecationWarning
\| * \| \| \| \| \| \| \| \| \| \|	Merge branch 'master' into bug/backend/b748/resolver-in-coro	Mark	2020-03-03	-1048/+1367
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Adding helpers to the Filtering whitelist	Leon Sandøy	2020-03-02	-0/+1
\| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Resolves an issue mentioned in https://github.com/python-discord/bot/issues/767, giving Helpers access to post invites and other things caught by the Filtering cog.
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #783 from python-discord/python38-migration	S. Co1	2020-03-01	-661/+301
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Migrate bot to Python 3.8
\| * \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into python38-migration	Leon Sandøy	2020-03-01	-2/+29
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #809 from python-discord/ban-grabify-urls	kwzrd	2020-03-01	-0/+24
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add grabify (IP logger) domains to banned domains
\| * \| \| \| \| \| \| \| \| \| \| \|	Add grabify (IP logger) domains to banned domains	Joseph	2020-03-01	-0/+24
\|/ / / / / / / / / / / /