aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | | | | fix: remove () from list of safe filename charsGravatar vcokltfre2021-04-30-1/+1
|/ / / / / / / / / / / / / /
* | | | | | | | | | | | | | Merge pull request #597 from python-discord/Enforce-image-processing-concurrencyGravatar ks1292021-04-30-413/+605
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move avatar commands into executors
| * \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into Enforce-image-processing-concurrencyGravatar ks1292021-04-30-268/+748
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | |
* | | | | | | | | | | | | | | Merge pull request #709 from chincholikarsalil/fix-grammatical-errorGravatar ChrisJL2021-04-27-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reworded/fixed grammatical error
| * | | | | | | | | | | | | | | Updated bot/utils/exceptions.pyGravatar Salil Chincholikar2021-04-27-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joe Banks <[email protected]>
| * | | | | | | | | | | | | | | Reworded/fixed grammatical errorGravatar Salil Chincholikar2021-04-27-1/+1
|/ / / / / / / / / / / / / / /
* | | | | | | | | | | | | | | Merge pull request #700 from python-discord/update-pr-templateGravatar Xithrius2021-04-24-20/+10
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reword PR Template
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into update-pr-templateGravatar Xithrius2021-04-24-3/+3
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | Merge pull request #702 from python-discord/fix-reddit-index-errorGravatar ChrisJL2021-04-23-3/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix errors when a subreddit has <5 posts.
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into fix-reddit-index-errorGravatar ChrisJL2021-04-23-2/+16
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Fix spelling of a user facing message in reddit cogGravatar ChrisJL2021-04-23-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Xithrius <[email protected]>
| * | | | | | | | | | | | | | | | Fix errors when a subreddit has <5 posts.Gravatar Chris2021-04-22-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a subreddit has <2 posts, the posts[1] check would fail with an IndexError. If the subreddit had less that 5 posts, then the k=5 check would also error. These changes harden the command for these edge cases.
| | * | | | | | | | | | | | | | | Merge branch 'main' into update-pr-templateGravatar Hassan Abouelela2021-04-23-2/+16
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Merge pull request #704 from python-discord/link-suppressingGravatar Hassan Abouelela2021-04-23-2/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | Suppress Links In Certain Commands
| * | | | | | | | | | | | | | | | Suppresses Links In CommandsGravatar Hassan Abouelela2021-04-23-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Suppresses links in certain commands that can echo back user input. Signed-off-by: Hassan Abouelela <[email protected]>
| * | | | | | | | | | | | | | | | Adds Link Suppressing HelperGravatar Hassan Abouelela2021-04-23-0/+8
|/ / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a helper to find and escape links in a message. Signed-off-by: Hassan Abouelela <[email protected]>
| * | | | | | | | | | | | | | | Reword Issues CommentGravatar Hassan Abouelela2021-04-23-1/+1
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Updates Punctuation In PR TemplateGravatar Hassan Abouelela2021-04-22-3/+3
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Rewords PR TemplateGravatar Hassan Abouelela2021-04-22-20/+10
|/ / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a rewording of the comments in the PR template, to hopefully clarify intentions. It also removes unneeded fields. Signed-off-by: Hassan Abouelela <[email protected]>
* | | | | | | | | | | | | | | Merge pull request #697 from ToxicKidz/fix_catifyGravatar Janine vN2021-04-21-19/+27
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed the .catify command producing UnboundLocal and Forbidden Errors
| * | | | | | | | | | | | | | | chore: use 'nickname' and 'display name' in the right places and use ↵Gravatar ToxicKidz2021-04-20-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | allowed_mentions
| * | | | | | | | | | | | | | | fix: Use name.replace not text.replaceGravatar ToxicKidz2021-04-20-3/+3
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | chore: lower the input to fine more catsGravatar ToxicKidz2021-04-20-3/+4
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | chore: Fix UnboundLocalError and discord.ForbiddenErrors in the catify commandGravatar ToxicKidz2021-04-20-17/+21
|/ / / / / / / / / / / / / / /
* | | | / / / / / / / / / / / Add Catify command (#694)Gravatar Dillon Runke2021-04-20-0/+83
| |_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joe Banks <[email protected]> Co-authored-by: hypergamer80 <[email protected]>
* | | | | | | | | | | | | | Merge pull request #673 from janine9vn/int-evalGravatar Matteo Bertucci2021-04-17-0/+435
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding Internal Eval to Sir Lancebot
| * \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into int-evalGravatar Matteo Bertucci2021-04-17-472/+730
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | |
* | | | | | | | | | | | | | | Merge pull request #686 from python-discord/remove-ffmpgGravatar Xithrius2021-04-16-87/+12
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | Remove ffmpg
| * | | | | | | | | | | | | | Remove un-used sound resourcesGravatar Chris2021-04-16-41/+0
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Remove ffmpg and discord.py voice extraGravatar Chris2021-04-16-46/+12
|/ / / / / / / / / / / / / /
| * | | | | | | | | | | | | Remove check for `exit` to reset contextGravatar Janine vN2021-04-12-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit removes the `exit` check if someone were to use this: `.int e exit` to clear the context. The check would prevent `.int e exit()` from restarting the bot container. With the `.int reset` and `.int exit` ability to clear the context the check for `exit` to clear the context isn't necessary.
| * | | | | | | | | | | | | Add constants for common string filenamesGravatar Janine vN2021-04-12-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a constant for the same filenames used in several locations. Because the now-a-constant string is used in several locations this will allow for it to be updated more easily down the line.
| * | | | | | | | | | | | | Ensure output will be robust for discord markdownGravatar Janine vN2021-04-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added in an extra `\n` at the end of the output. Sometimes discord won't properly format the codeblock in the triple ` is not on a newline. This changes ensures that it should.
| * | | | | | | | | | | | | Change command help formatGravatar Janine vN2021-04-11-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `.int` with nothing else now uses the `invoke_help_command()` utility that formats the help command much more nicely than the default version
| * | | | | | | | | | | | | Update docstring for resetGravatar Janine vN2021-04-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated the docstring for `reset` to provide accurate information as to what the command does.
| * | | | | | | | | | | | | Remove unused codeblock regexGravatar Janine vN2021-04-11-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the regex sufficiently stolen from snekbox and confirmed to work, the original codeblock regex has been removed.
| * | | | | | | | | | | | | Correct logger nameGravatar Janine vN2021-04-11-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed the initialization of the logging to pull dynamically so it can actually log correctly.
| * | | | | | | | | | | | | Correct prefix usage in a doctstring Gravatar Janine vN2021-04-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Corrects the prefix for the a command in the docstring to use Lancebot's prefix. Co-authored-by: Matteo Bertucci <[email protected]>
| * | | | | | | | | | | | | Blind Fixes at LintingGravatar janine9vn2021-04-10-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both my pre-commit and flake8 runs are telling me everything is fine and it's all passed. Github actions is saying otherwise but isn't saying *where*. So here I am with useless linting commits.
| * | | | | | | | | | | | | Removed rogue variableGravatar janine9vn2021-04-10-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm better than this I swear. I can lint before I commit. Don't tell lemon.
| * | | | | | | | | | | | | Update codeblock regexGravatar janine9vn2021-04-10-1/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The snekbox implementation of the codeblock regex was incorporated. This now correctly parses the `code` and ``code`` markdown discord allows. You can also use multiple code blocks with text interrupting it and it will process the different code blocks as one continuous code block.
| * | | | | | | | | | | | | Change namesGravatar janine9vn2021-04-10-395/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These were missed in a previous commit. It's a simple name change from the original files to better align with Sir Lancebot.
| * | | | | | | | | | | | | Realigned to SirLancebot StructureGravatar janine9vn2021-04-09-0/+405
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code for rattlesnakes's internal eval was aligned to Sir Lancebot's structure. It was mostly renaming rattlesnake to bot and changing how some of the imports were setup as. It also included changing the __init__.py to match the Sir Lancebot cog structure. Additionally, the whitelist check has been significantly simplified to only be a role check for the admin role. The rattlesnake implementation had a more robust `in_whitelist` decorator, so it may be worth investigating adding that in if we see fit. For now, it's a simple `with_role` decorator check. The name of the cog file itself was changed to include an underscore to sidestep what I think was a namespace collision that would prevent the setup function from properly running.
| * | | | | | | | | | | | | Add helpers for internal evalGravatar janine9vn2021-04-09-0/+243
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cutting over the rattlesnake helpers specifically for internal_eval. I am mirroring the rattlesnake structure as much as I can initially to ensure basic functionality before migrating functions to fit more within Sir Lancebot's file structure.
| * | | | | | | | | | | | | Cutover of rattlesnake to lancebotGravatar janine9vn2021-04-09-0/+152
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an initial cutover of the rattlesnake internal eval to Sir Lancebot. This commit by itself will not work. This is a simple drop in of rattlesnake code so there is context as to what has changed and why.
| | * | | | | | | | | | | | Add non-underscore aliases for lazy people like me :)Gravatar Chris2021-04-30-1/+1
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Consistant use of double quotes and tuples in avatar_mod cogGravatar Chris2021-04-30-11/+10
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Rename pfp_modify cog and group to avatar_mod for clarityGravatar Chris2021-04-30-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added in aliases for ease of use too.
| | * | | | | | | | | | | | Escape invalid filename chars before saving.Gravatar Chris2021-04-22-16/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This would cause an issue with the embed not embedding the image due to a filename mismatch. This has been implemented by escaping all invalid filename characters before saving the file.
| | * | | | | | | | | | | | Fix captitilization in docstringGravatar Chris2021-04-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: LXNN <[email protected]>