aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | | | | * | | | | | | Removed `zen` tag due `!zen` command exist.Gravatar Karlis S2020-03-23-20/+0
| |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | |
| | | | | | | | | * | | | Clarify docstring for token check functionGravatar S. Co12020-03-23-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | | | | * | | | Update token filter logging to match expanded detectionGravatar S. Co12020-03-20-12/+18
| |_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Log message still used the first regex result (re.search) rather than the expanded approach (re.findall) recently added.
* | | | | | | | | | | | Merge pull request #830 from ks129/bug-fixesGravatar Mark2020-03-16-1/+4
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | ModLog Channel Update formatting fix.
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug-fixesGravatar Mark2020-03-16-0/+80
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #828 from python-discord/feat/test/765/command-shadowingGravatar Mark2020-03-16-0/+80
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | | | Test for command name and alias shadowing
| * | | | | | | | | | | | Merge branch 'master' into feat/test/765/command-shadowingGravatar Mark2020-03-16-106/+937
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Cog tests: comment some code for clarificationGravatar MarkKoz2020-03-16-0/+3
| | | | | | | | | | | | |
| * | | | | | | | | | | | Cog tests: fix error on import due to discord.ext.tasks.loopGravatar MarkKoz2020-03-13-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tasks extensions loop requires an event loop to exist. To work around this, it's been mocked.
| * | | | | | | | | | | | Cog tests: fix leading space in aliases without parentsGravatar MarkKoz2020-03-13-1/+1
| | | | | | | | | | | | |
| * | | | | | | | | | | | Cog tests: add a test for duplicate command names & aliasesGravatar MarkKoz2020-03-13-0/+17
| | | | | | | | | | | | |
| * | | | | | | | | | | | Cog tests: fix duplicate commands being yieldedGravatar MarkKoz2020-03-13-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | discord.py yields duplicate Command objects for each alias a command has, so the duplicates need to be removed on our end.
| * | | | | | | | | | | | Cog tests: fix nested modules not being foundGravatar MarkKoz2020-03-13-9/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rename `walk_extensions` to `walk_modules` because some extensions don't consist of a single module
| * | | | | | | | | | | | Cog tests: fix duplicate cogs being yieldedGravatar MarkKoz2020-03-13-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Have to check the modules are equal to prevent yielding imported cogs.
| * | | | | | | | | | | | Cog tests: fix Cog type check in `walk_cogs`Gravatar MarkKoz2020-03-13-3/+3
| | | | | | | | | | | | |
| * | | | | | | | | | | | 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.