aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge branch 'master' into feat/test/765/command-shadowingGravatar Mark2020-03-16-106/+937
|\
| * 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.
* | | | | 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
|/ / / /
* | | | 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.