aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Don't call `on_command_error` when a check errors.sim_matchesGravatar Qwerty-1332021-09-15-5/+4
|
* Merge pull request #864 from NIRDERIi/hanukkah_cmd_monthGravatar Xithrius2021-09-09-1/+1
|\ | | | | Added Hanukka command in November month.
| * Added Hanukka command in November month.Gravatar NIRDERIi2021-09-08-1/+1
|/ | | This is a solution for the #862 issue, which says the Hanukka holiday can also be in November and not only in December.
* Wikiguess Game (#618)Gravatar Shivansh-0072021-09-07-217/+299
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * (trivia-quiz): Add Wikipedia Guess Game This commit also moves all the 'dynamic' question generator to a separate class. Closes: #446 * (trivia quiz): Use tuples for command aliases * (trivia quiz): Edit congratulations message * (trivia quiz): Use default dict for storing player scores * (trivia quiz): 'done_question' to 'done_questions' * Add space after 'Congratulations' word * Use classmethods for dynamically generating questions * Don't add wiki category if max error fetches hit If the task hit max error fetches, which is 3 currently, it would remove wikipedia from listed categories and not add it to loaded questions. If it doesn't hit max fetches, then it adds them. * Don't hardcode the number of questions in RULES * Add information field only if it exists * Add "cs" and "python" categories to the `.quiz` command * add 30 questions each under the categories "cs" and "python" add the two categories into the code and modify the starting phase Co-authored-by: Xithrius <[email protected]> Co-authored-by: ToxicKidz <[email protected]> * refactor: Use yesterday's most read to make trivia questions Since random wikipedia article guess questions weren't really "knowledgeable", no one could really guess it or gain any "good" knowledge from them, so after asking wookie (this commits mentions his review comments above also), I decided to use these. * refactor: Logic to remove pronounciations from question Co-authored-by: wookie184 <[email protected]> * fix: Set to correct question limit If the number of questions are less then the default limit which can happen in the case of wikipedia guess game as it is dependent on the most read articiles on wikipedia, it would create a infinite loop sending us into infinite amount of errors, so let's prevent that, thanks wookie * chore: Add comment for d5f8205 change * refactor: Remove double mention of dataclass in quizentry * chore: Use r"" over noqa We can use r"", a raw string, here to make it clear that \* and \s aren't supposed to be handled as escape sequences and just use the "raw string". * fix: Correct off by one bug Originally, before this commit, we checked the number of questions left by comparing `len(done_questions) > self.question_limit`, so question limit had to be 1 since if it wasn't we would compare 7 > 7, which would be false and then it would send another question. To correct this bug, we now use == comparision on the two, so if the number of done questions is same as the question limit it means that the round is over. I have changed the relevant parts of the code to reflect this change i.e. where-ever we did +-1 due to the off by one bug. * refactor: Noramlize the title to remove all punctuations Since the title can sometimes contain punctuations making it very difficult to get the matching answer to the question, we originally removed all such questions. This took the question count down :( and wasn't an effective way. Therefore now we keep them but as normalized, yay! I have also updated the code documentation to make the process much clearer to anyone reading the "normalizing" code section of the wiki questions generator. * refactor: Keep answers as a list & not ",".join() Wasn't fitting in character limit so shorterned it ^^ lol. Okay, getting to the point, this mentions fix error's comment of making quiz entry except the answers as a list and not as a string which could a comma joined list. The same structure was in the json resource, where multiple answers where joined with commas. This didn't allow you to use commas in answers. So I went ahead and did a bit more than requested to change the json structure and make `answers` a list. Also now all questions are in the form of the quiz entry to keep it same through out the code and var tolerance has become a valid param in QuizEntry, this is done because it was differing between questions, if not needed this would make the process to add `var_tol` as a argument to the json easier. And that's it! Co-authored-by: Objectivitix <[email protected]> Co-authored-by: Xithrius <[email protected]> Co-authored-by: ToxicKidz <[email protected]> Co-authored-by: wookie184 <[email protected]>
* CI: check license compatibility (#855)Gravatar Matteo Bertucci2021-09-06-109/+172
| | | | | | | | | | | | | | | | | * CI: check for license compatibility Port of https://github.com/python-discord/bot/pull/1779 --- Since our project is licensed under the MIT License, we can't be using any incompatible license-wise dependencies in our project, such as GPL. This commit adds a step to the CI lint phase that will use pip-licenses to verify that all the installed packages are part of an ALLOWED_LICENSE variable. This variable is currently set to be every license we currently use. We opted to use an allowlist instead of a denylist to make sure that new licenses are reviewed by hand and added to that variable. Co-authored-by: ChrisJL <[email protected]> Co-authored-by: Xithrius <[email protected]> Co-authored-by: Vivaan Verma <[email protected]>
* Merge pull request #850 from Numerlor/async-redisache-logsGravatar ChrisJL2021-09-06-0/+1
|\ | | | | Set logging level for async-rediscache to warning
| * Merge branch 'main' into async-redisache-logsGravatar ChrisJL2021-09-06-103/+86
| |\ | |/ |/|
* | Merge pull request #853 from python-discord/post-restructure-fixGravatar Xithrius2021-09-05-1/+1
|\ \ | | | | | | Post re-structure fixs
| * | Correct an import path in AOC helpersGravatar Chris Lovering2021-09-05-1/+1
| | |
| * | Add a missing __init__.py causing event cogs not to loadGravatar Chris Lovering2021-09-05-0/+0
|/ /
* | Merge pull request #851 from python-discord/lance-restructureGravatar ChrisJL2021-09-05-103/+86
|\ \ | | | | | | Restructure Sir Lancebot
| * \ Merge branch 'main' into lance-restructureGravatar ChrisJL2021-09-05-1/+1
| |\ \ | |/ / |/| |
| * | Update rogue paths to new pathsGravatar Janine vN2021-09-05-12/+12
| | |
| * | Update CODEOWNERS to reflect file movesGravatar Janine vN2021-09-05-2/+3
| | |
| * | Move game and fun commands to Fun folder, fix ddgGravatar Janine vN2021-09-05-36/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This moves all the fun commands and games into the fun folder. This commit also makes changes to the duck_game. It was setting a footer during an embed init, which is no longer possible with the version of d.py we use. Additionally, an issue with editing an embed that had a local image loaded. The workaround for the time being is to update the message, not the embed.
| * | Move snakes commands into fun folderGravatar Janine vN2021-09-05-11/+11
| | |
| * | Move practical functions into utilities folderGravatar Janine vN2021-09-05-5/+5
| | | | | | | | | | | | | | | | | | Separates out the useful/practical seasonal bot features from the evergreen folder into a "utilities" folder. Adjusts the paths to resources to reflect the folder move.
| * | Move internal eval and rename utils to coreGravatar Janine vN2021-09-05-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Part of this restructure involves splitting out the massive evergreen folder into a `fun` folder and then a `utilities` folder. To help with that we've rename the `util` folder to `core`. The core functions to run the bot have been moved into this folder. `.source`, `.ping`, and `.int e` have been moved into this folder.
| * | Update paths to new resource linksGravatar Janine vN2021-09-04-5/+5
| | | | | | | | | | | | | | | | | | Additionally, this commit fixes an error with the pridepfp command. The avatar image now uses discord.py's v.20 avatar.url instead of avatar_url
| * | Move Hanukkah to Holidays folderGravatar Janine vN2021-09-04-0/+0
| | | | | | | | | | | | | | | Hannukah was previously in the Christmas folder, which was weird. This now moves it to its own folder under Holidays.
| * | Move Pride to Holidays folderGravatar Janine vN2021-09-04-4/+4
| | | | | | | | | | | | | | | Moves the Pride features to the Holidays folder. Corrected the paths to reflect this change.
| * | Move Valentines to Holidays folderGravatar Janine vN2021-09-04-8/+8
| | | | | | | | | | | | | | | Moves the valentine's day features to the holidays folders. Corrected the paths to reflect the folder moves.
| * | Move Halloween to Holidays folderGravatar Janine vN2021-09-04-7/+9
| | | | | | | | | | | | | | | Moves all the hallowen features to the holidays folder. Also updates the paths to reflect the folder moves.
| * | Move Easter to Holidays FolderGravatar Janine vN2021-09-04-10/+10
| | | | | | | | | | | | | | | | | | This moves the easter seasonal features into a more cohesive holidays/easter folder. Additionally, this splits out earth day into its own holiday folder.
| * | Move AoC and Hacktoberfest into events folderGravatar Janine vN2021-09-04-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moves Advent of Code and Hacktoberfest into an events folder. Although these are roughly associated with holidays, they are standalone events that we have participated in in the past. Therefore they're being moved to an events folder separate from the "fun" or "holidays" folders.
| | * Set logging level for async-rediscache to warningGravatar Numerlor2021-09-05-0/+1
| |/ |/| | | | | Logs below warnings aren't relevant when working on the bot
* | Merge pull request #845 from python-discord/Pin-platform-in-DockerfileGravatar Johannes Christ2021-09-04-1/+1
|\ \ | |/ |/| Pin platform in Dockerfile
| * Pin platform in DockerfileGravatar Chris Lovering2021-09-04-1/+1
|/ | | | | | Some of our deps don't have wheels for atm processors. With Mac's M1 chips becomming more common, we should make it easier for those users to build our environments.
* Merge pull request #844 from python-discord/jb3/update-k8s-pathGravatar Joe Banks2021-09-04-1/+1
|\ | | | | Update manifest path
| * Update in accordance with python-discord/kubernetes#95Gravatar Joe Banks2021-09-04-1/+1
|/
* Merge pull request #833 from brad90four/teapot-supportGravatar ChrisJL2021-09-03-46/+50
|\ | | | | Teapot support - Continues PR #610
| * Merge branch 'main' into teapot-supportGravatar ChrisJL2021-09-03-502/+531
| |\ | |/ |/|
* | Merge pull request #839 from python-discord/android-codeblock-fixGravatar Xithrius2021-09-03-43/+47
|\ \ | | | | | | Fix android codeblock rendering inconsistency
| * \ Merge branch 'main' into android-codeblock-fixGravatar Xithrius2021-09-03-272/+271
| |\ \ | |/ / |/| |
* | | Merge pull request #802 from python-discord/decorator-factory/typehints-fixGravatar Xithrius2021-09-02-266/+263
|\ \ \ | | | | | | | | | | | | | | | | * Fix type annotations Co-authored-by: decorator-factory <[email protected]>
| * \ \ Merge branch 'main' into decorator-factory/typehints-fixGravatar Xithrius2021-09-02-6/+8
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #838 from ↵Gravatar ChrisJL2021-09-01-6/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/Use-permissions_for-over-permissions_in Use permissions_for over permissions_in
| * | | | Use permissions_for over permissions_inGravatar Chris Lovering2021-09-01-6/+8
|/ / / / | | | | | | | | | | | | `discord.Member.permissions_in()` was removed in d.py 2.0 in favour of using `discord.Channel.permissions_for()` everywhere.
| * | | More explicit importing for `Bot`Gravatar Xithrius2021-09-02-9/+9
| | | |
| * | | A list of emojisGravatar Xithrius2021-09-02-1/+1
| | | |
| * | | Removed more not-needed `None` returnsGravatar Xithrius2021-09-02-3/+3
| | | |
| * | | Removed unecessary `None` annotationGravatar Xithrius2021-09-02-2/+1
| | | |
| * | | Added documentation for tupleGravatar Xithrius2021-09-02-1/+1
| | | |
| * | | Removed depricated `typing.Match`Gravatar Xithrius2021-09-02-2/+2
| | | |
| * | | Removed unecessary annotations.Gravatar Xithrius2021-09-01-4/+3
| | | |
| * | | Removed None return annotation for any __init__Gravatar Xithrius2021-09-01-19/+19
| | | |
| * | | Union item with None to Optional with item.Gravatar Xithrius2021-09-01-10/+10
| | | |
| * | | Fix type annotationsGravatar decorator-factory2021-08-31-228/+227
|/ / /
| * | Improved consistency for codeblocks to end with a newlineGravatar wookie1842021-09-01-30/+34
| | |
| * | Start and end codeblocks with newlines to avoid android rendering them inlineGravatar wookie1842021-09-01-13/+13
|/ /