aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* CI: install pipenv to user siteGravatar MarkKoz2020-03-09-5/+5
| | | | | | | | | | Some of pipenv's dependencies overlap with dependencies in the Pipfile. When installing from the Pipfile, any dependencies already present in the global site will not be installed again to the user site, and thus will not be cached. Therefore, pipenv is installed to the user site to ensure all dependencies get cached. * Move PATH prepend step before pipenv invocation
* CI: invalidate cachesGravatar MarkKoz2020-03-09-6/+6
|
* CI: prepend py user base to PATHGravatar MarkKoz2020-03-09-0/+3
|
* CI: cache the Python user base dirGravatar MarkKoz2020-03-09-3/+5
|
* CI: cache Python dependenciesGravatar MarkKoz2020-03-09-1/+14
| | | | | | | | | Reduces frequency of using pipenv to install dependencies in CI. Works by caching the entire Python directory. Only a full cache hit will skip the pipenv steps; a partial cache hit will still be followed by using pipenv to install from the pipfiles. * Disable pip cache
* CI: make env vars used for coverage into pipeline variablesGravatar MarkKoz2020-03-09-1/+7
| | | | Makes the script for the coverage step cleaner.
* 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
| | |
| * | Merge branch 'master' into roles-command-paginationGravatar S. Co12020-03-03-735/+507
| |\ \
| * | | !roles Command Test: Applied !roles command changesGravatar Karlis. S2020-03-01-3/+2
| | | |
| * | | !roles Command: Added pagination (LinePaginator), moved roles amount to ↵Gravatar Karlis. S2020-03-01-8/+7
| | | | | | | | | | | | | | | | title (was before in footer).
| | | * Raise BadArgument if no string match.Gravatar scragly2020-03-05-0/+3
| | | |
| | | * Re-use embed, use command converter, raise BadArgument.Gravatar scragly2020-03-05-37/+22
| | | |
| | | * Merge branch 'master' into zen-commandGravatar scragly2020-03-05-2673/+5311
| | | |\ | |_|_|/ |/| | |
* | | | Merge pull request #814 from python-discord/antimalware-loggingGravatar Mark2020-03-04-17/+24
|\ \ \ \ | | | | | | | | | | Expand Antimalware User Feedback & Logging
| * \ \ \ Merge branch 'master' into antimalware-loggingGravatar Mark2020-03-04-133/+97
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #816 from ↵Gravatar Shirayuki Nekomata2020-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-warningGravatar Shirayuki Nekomata2020-03-04-128/+85
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #811 from python-discord/feat/deps/o138/pre-commit-hooksGravatar scragly2020-03-04-128/+85
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add more pre-commit hooks and run them in CI
| * \ \ \ \ \ Merge branch 'master' into feat/deps/o138/pre-commit-hooksGravatar scragly2020-03-04-33/+102
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |