aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* 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.
* 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
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Update code for pep8-naming complianceGravatar S. Co12020-03-03-6/+6
| | | | | | |
| * | | | | | Remove CaseInsensitiveDictGravatar S. Co12020-03-03-94/+0
| | | | | | | | | | | | | | | | | | | | | This was added by the now-removed Snake cog & is not used elsewhere on bot.
| * | | | | | Add pep8-naming & relockGravatar S. Co12020-03-03-14/+28
| | | | | | |
| * | | | | | CI: mock the pipenv binaryGravatar MarkKoz2020-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 cacheGravatar MarkKoz2020-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 scriptGravatar MarkKoz2020-03-03-4/+12
| | | | | | |
| * | | | | | Add more pre-commit hooksGravatar MarkKoz2020-03-02-10/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove trailing whitespaces * Specify error code for a noqa in the free command
| | * | | | | Use raw strings for docstrings with forward slashesGravatar Sebastiaan Zeeff2020-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-dmGravatar scragly2020-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 botGravatar S. Co12020-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-loggingGravatar S. Co12020-03-03-1076/+1451
| | |\ \ \ | |_|/ / / |/| | | |
* | | | | Merge pull request #750 from python-discord/bug/backend/b748/resolver-in-coroGravatar Mark2020-03-03-31/+93
|\ \ \ \ \ | |/ / / / |/| | | | Create AsyncResolver inside a coroutine to avoid DeprecationWarning
| * | | | Merge branch 'master' into bug/backend/b748/resolver-in-coroGravatar Mark2020-03-03-1048/+1367
| |\ \ \ \ | |/ / / / |/| | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-26 16:19:02 +0000