aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | Cog tests: add a function to yield all commandsGravatar MarkKoz2020-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 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
| | |\ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | |
| * | | | | | | | | convert get_tags() method to staticmethodGravatar RohanJnr2020-03-12-1/+2
| |/ / / / / / / /
| * | | | | | | | remove repetitive file searchGravatar RohanJnr2020-03-12-3/+2
| | | | | | | | |
| * | | | | | | | Update ytdl tag to the new YouTube ToSGravatar Matteo Bertucci2020-03-12-5/+4
| | | | | | | | |
| * | | | | | | | Update tag files for new linting hooksGravatar S. Co12020-03-04-59/+59
| | | | | | | | |
| * | | | | | | | Merge branch 'master' into tags_overhaulGravatar S. Co12020-03-04-921/+690
| |\ \ \ \ \ \ \ \
| * | | | | | | | | Convert "get_tags()" and "_get_tag()" to sync functionsGravatar RohanJnr2020-03-04-10/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "get_tags()" and "_get_tag()" functions need not be async as we are no longer doing any API call but instead reading from local files.
| * | | | | | | | | Use "pathlib" instead of "os" module and context managerGravatar RohanJnr2020-03-04-11/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The pathlib module simplifies opening and reading files, hence the os module and the context manager are no longer used.
| * | | | | | | | | Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵Gravatar RohanJnr2020-02-29-895/+2636
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tags_overhaul
| | * \ \ \ \ \ \ \ \ Merge branch 'master' into tags_overhaulGravatar Rohan2020-02-29-895/+2636
| | |\ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Caching all the tags when the bot has loaded(caching only once) insted of ↵Gravatar RohanJnr2020-02-29-21/+18
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | caching it after the tags command is used.
| * | | | | | | | | | Re-corrected the lines which I had changed by mistakeGravatar RohanJnr2020-02-29-1/+2
| | | | | | | | | | |
| * | | | | | | | | | added white spaces on statements before bullet points for proper rendering ↵Gravatar RohanJnr2020-02-29-75/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of points on github
| * | | | | | | | | | Added all the tag files in resources and modified cogs/tags.py file to ↵Gravatar RohanJnr2020-02-29-84/+919
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
| | | | | | | | | | * CI: remove support for partial cache hitsGravatar MarkKoz2020-03-25-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Partial hits may cause issues when packages get removed. The cache will get bloated with packages which are no longer needed. They will keep accumulating as more packages get removed unless the cache is unused for 7 days and gets automatically deleted by Azure Pipelines. Lingering packages are also a potential cause for conflicts (e.g. unused package x depends on package y==4.0 and useful package z depends on y==5.0). Removing support for partial hits means all dependencies will be installed whenever a single dependency changes.
| | | | | | | | | | * CI: rename UsePythonVersion taskGravatar MarkKoz2020-03-11-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "python" is a shorter and clearer name.
| | | | | | | | | | * CI: don't do a user install for pre-commit venvGravatar MarkKoz2020-03-09-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevents the following error: Can not perform a '--user' install. User site-packages are not visible in this virtualenv.
| | | | | | | | | | * 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