aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | Cog tests: add a function to get all qualified names for a cmdGravatar MarkKoz2020-03-13-0/+8
| | | | | | | | | | | |
| * | | | | | | | | | | Cog tests: add a function to get all cogsGravatar MarkKoz2020-03-13-0/+7
| | | | | | | | | | | |
| * | | | | | | | | | | Cog tests: add a function to get all extensionsGravatar MarkKoz2020-03-13-0/+11
| | | | | | | | | | | |
| * | | | | | | | | | | Cog tests: add a function to get all commandsGravatar MarkKoz2020-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 testsGravatar MarkKoz2020-03-09-0/+7
| | |_|_|_|_|/ / / / / | |/| | | | | | | | |
| | * | | | | | | | | Update explanation comment so it explains what happensGravatar S. Co12020-03-16-2/+3
| | | | | | | | | | |
| | * | | | | | | | | (Mod Log): Added comment about channel update formatting change.Gravatar Karlis S2020-03-16-0/+2
| | | | | | | | | | |
| | * | | | | | | | | (Mod Log): Fixed case when `on_guild_channel_update` old or new value is ↵Gravatar ks1232020-03-16-1/+1
| |/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | empty and with this message formatting go wrong.
* | | | | | | | | | Merge pull request #803 from RohanJnr/tags_overhaulGravatar Shirayuki Nekomata2020-03-16-106/+937
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | | Removed API dependency for Tags
| * | | | | | | | | Fixed tag search via contents, any keywords.Gravatar Shirayuki Nekomata2020-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-asyncGravatar RohanJnr2020-03-15-2/+2
| | | | | | | | | |
| * | | | | | | | | Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵Gravatar RohanJnr2020-03-15-9/+15
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tags_overhaul
| | * | | | | | | | | Tags: add restrictions 1 & 9 from YouTube ToS to ytdl tagGravatar MarkKoz2020-03-14-2/+6
| | | | | | | | | | |
| | * | | | | | | | | Tags: use constant for command prefix in embed footerGravatar MarkKoz2020-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-asyncGravatar RohanJnr2020-03-15-1/+1
| |/ / / / / / / / /
| * | | | | | | | | Remove line that calls get_tags() methodGravatar RohanJnr2020-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 ↵Gravatar RohanJnr2020-03-12-22/+200
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tags_overhaul
| | * \ \ \ \ \ \ \ \ Merge branch 'master' into tags_overhaulGravatar Rohan2020-03-12-22/+200
| | |\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #822 from python-discord/bug/mod/792/null-attachmentsGravatar Eivind Teig2020-03-09-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | ModLog: fix posting null attachments for deleted message logs
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug/mod/792/null-attachmentsGravatar Eivind Teig2020-03-09-6/+103
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #825 from python-discord/token-detection-fixGravatar Eivind Teig2020-03-09-3/+10
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Refactor token detection to check all potential substrings in message
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into token-detection-fixGravatar Eivind Teig2020-03-09-3/+93
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #826 from python-discord/voteGravatar Eivind Teig2020-03-09-0/+21
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement vote command.
| * | | | | | | | | | | | | Disambiguate codepoint value.Gravatar scragly2020-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.Gravatar scragly2020-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 voteGravatar scragly2020-03-09-2/+2
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | |
* | | | | | | | | | | | | | Merge pull request #824 from python-discord/antimalware-fixGravatar Mark2020-03-08-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix filtered extension string out of scope for antismalware log message
| * \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into antimalware-fixGravatar Mark2020-03-08-1/+70
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Fix filtered extension string out of scope for log messageGravatar S. Co12020-03-08-2/+2
| | |_|_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | * Fix typo in file extensions list comprehension
| | * | | | | | | | | | | | Implement vote command.Gravatar scragly2020-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 ↵Gravatar Mark2020-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`Gravatar Shirayuki Nekomata2020-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.Gravatar Shirayuki Nekomata2020-03-08-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | | | | | | | | | | | Refactored dense codes, removed obvious type hint.Gravatar Shirayuki Nekomata2020-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 ↵Gravatar Shirayuki Nekomata2020-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 contentsGravatar Shirayuki Nekomata2020-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 ↵Gravatar Shirayuki Nekomata2020-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 messageGravatar S. Co12020-03-08-3/+10
|/ / / / / / / / / / /
| * / / / / / / / / / ModLog: fix posting null attachments for deleted message logsGravatar MarkKoz2020-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-tweaksGravatar Eivind Teig2020-03-06-2/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Pre-commit tweaks
| * | | | | | | | | | Remove excludes from pre-commitGravatar MarkKoz2020-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 hooksGravatar MarkKoz2020-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-commandGravatar scragly2020-03-05-2/+86
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | implement !zen command.
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into zen-commandGravatar scragly2020-03-05-11/+9
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #807 from ks129/roles-command-paginationGravatar scragly2020-03-05-11/+9
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Added Pagination to !roles command
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into roles-command-paginationGravatar scragly2020-03-05-150/+121
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | (Information Cog, !roles command test): Removed 's' at end of "Total 1 ↵Gravatar ks1232020-03-05-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | role(s)" due changes in command.
| * | | | | | | | | | | (Information Cog, !roles command): Added 's' to end of 'role' only if there ↵Gravatar ks1232020-03-05-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | is more then 1 role.
| * | | | | | | | | | | (Information Cog, !roles command test): Applied empty parameter change.Gravatar ks1232020-03-05-1/+1
| | | | | | | | | | | |
| * | | | | | | | | | | (Information Cog, !roles command): Added empty parameter to pagination (False)Gravatar ks1232020-03-05-1/+1
| | | | | | | | | | | |