aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge pull request #899 from python-discord/fix-issue-898Gravatar Hassan Abouelela2021-10-17-6/+10
|\ | | | | Set `AI.user` to @Sir Lancebot
| * Merge branch 'main' into fix-issue-898Gravatar ChrisJL2021-10-17-17/+572
| |\ | |/ |/|
* | Merge pull request #914 from python-discord/issue-command-fixesGravatar Karlis Suvi2021-10-17-5/+14
|\ \ | | | | | | Fix bugs in `.issue` command & add aliases
| * | Fix bugs in `.issue` command & add aliasesGravatar TizzySaurus2021-10-17-5/+14
|/ / | | | | | | | | | | - Now requires at least one issue/PR - No longer continues to send issues/PRs when there's too many listed in the invocation - Added plural aliases (`.issues` and `.prs`)
* | Merge pull request #889 from python-discord/fix-pascal-triangle-imageGravatar wookie1842021-10-14-1/+1
|\ \ | | | | | | Change pascal's triangle image
| * \ Merge branch 'main' into fix-pascal-triangle-imageGravatar TizzySaurus2021-10-14-117/+18545
| |\ \ | |/ / |/| |
* | | Add support to query AoC results in respect of days and stars (#857)Gravatar D0rs4n2021-10-14-7/+119
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add support to query AoC results in respect of days and stars From now on the AoC leaderboard command accepts a total of 2 optional arguments a day and star string (eg.: 1-2, for the second star of the first day) and a number of results they would like to see, with a total maximum of 15. This commit also introduces a few minor fixes in the AoC helper. * Improve overall code consitency in the AoC event Cog and helpers * Improve indenting and code consistency in the AoC cog * Improve code transparency in the AoC helpers * Patch various inconsistencies in the AoC cog and helpers * Migrate AoC Day and Star statistics filtering to Dropdowns From now on when the AoC leadearboard command is used with the DayAndStar argument(bool) the bot will send a View with two dropdowns and a button to Fetch the data based on the value of the Dropdowns. * Improve code and comment consistency in the AoC views and helpers * Patch logic errors, improve consistency in the AoC cog and view. * Add support to delete view from the message after timeout in the AoC cog * Move the day_and_star logic out of the typing context manager in the AoC cog * Revert season-locker in the AoC cog * Improve overall code transparency and indenting in the AoC cog and views * Remove unnecessary returns in the AoC cog and view
* | | Merge pull request #908 from onerandomusername/patch-2Gravatar Karlis Suvi2021-10-14-1/+2
|\ \ \
| * | | Fix GH-907Gravatar aru2021-10-13-1/+2
|/ / / | | | | | | Issues can have empty bodies, in this case GitHub doesn't include the key in the API response
* | | Challenges (#860)Gravatar Shom7702021-10-13-1/+434
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * beginning commit creating the base of the hangman, code needs to be linted in the future * updated words list * adding images to show the hangman person * added images, though it is a bit laggy * replacing images with discord attachment urls * adding error if filters aren't found * fixing typo in ``filter_not_found_embed`` * final lints + removing `mode` parameter as it renders useless * linting flake8 errors * adding newline at the end of `top_1000_used_words.txt` * minor change to filter message * beginning commit -- trying to add bs4 to pyproject.toml, though it is currently failing * kata information section done, ready for issue * fixing bugs with the query not being fully picked up, also allowing query only with no kyu * fixing bug where user cannot leave all arguments blank * typo - forgot unary before the level within the `language and not query` if statement * changing to random kata chosen * ensuring that if the user provides a query that won't work, that it won't error out * limiting choice to smaller numbers if a query was provided, so the user gets what they want * improving hangman docstring * removing `bot/resources/evergreen/hangman` directory as file attachments are used * replacing single quotes with double quotes, to adhere to the style guide. * fixing style inconsistencies and other problems with how the code looks - as per requested by Objectivix * fixing `IMAGES` style inconsistency * adding trailing commas and switching to `Colours` for consistency * adding trailing commas and switching to `Colours` for consistency * fixing the remnants of non-trailing commas and allowing specification for single player vs mulitplayer * removing all 2 letter words from the hangman word choosing and removing words that @Objectivix found that shouldn't be in the list of words * removing some inappropriate words from the txt file * Adding space for grammatical errors Co-authored-by: ChrisJL <[email protected]> * changing two periods to a full stop & wrapping try and except block to only the part that can raise it * using negative replies instead along with fixing grammatical errors in the sentence * removing words that could be considered inappropirate * removing `TOP_WORDS_FILE_PATH` and making `ALL_WORDS` a global variable. * error handling * fixing the overcomplication of the bs4 portion * adding button and dropdowns to the challenges command * more specific docstring * more specific docstring * finishing dropdowns/buttons * putting the dropdown on top of the link button * replacing ' with a double quote for some strings * Removing more words The words removed shouldn't really belong here * Update bot/exts/utilities/challenges.py Co-authored-by: Bluenix <[email protected]> * replacing mapping_of_images with IMAGES and other fixes * Dedenting Co-authored-by: Bluenix <[email protected]> * Improving tries logic Co-authored-by: Bluenix <[email protected]> * Updating `positions` list to set Co-authored-by: Bluenix <[email protected]> * Updating setup docstring Co-authored-by: Bluenix <[email protected]> * Updating comment in callback function of the dropdown Co-authored-by: Bluenix <[email protected]> * fixing too many blank lines * Hardcode dictionary Co-authored-by: Bluenix <[email protected]> * restructuring * fixing errors * Remove unnecessary comments Co-authored-by: Bluenix <[email protected]> * Remove unnecessary comments Co-authored-by: Bluenix <[email protected]> * Improve comment explanation Co-authored-by: Bluenix <[email protected]> * Remove redundant extra membership test Co-authored-by: Bluenix <[email protected]> * Removing verbose variable definition Co-authored-by: Bluenix <[email protected]> * Redundant list Co-authored-by: Bluenix <[email protected]> * Shorten 'social distancing' (too many separations) between related lines Co-authored-by: Bluenix <[email protected]> * improving docstring in `kata_id` * sending embed if error occurs with api or bs4, also hardcoding params dictionary * Better comments Co-authored-by: Bluenix <[email protected]> * better docstring Co-authored-by: Bluenix <[email protected]> * Removing f-string inception and replacing it with more readable code Co-authored-by: Bluenix <[email protected]> * More specific docstring Co-authored-by: Bluenix <[email protected]> * Removing redundant comments Co-authored-by: Bluenix <[email protected]> * Fixing linting errors * mapping of kyu -> constant * adding trailing comma * specific comment regarding where colors are from for `MAPPING_OF_KYU` * changing name to link too along with link button * adding ellipsis to make it more clear for `Read more` * removing redundant sentences from all docstrings of embed creator functions * fixing unboundlocalerror due to kata_url only being defined under a certain condition * only allowing supported languages on codewars.com * fixing url glitch with embed * Delete hangman.py * Delete top_1000_used_words.txt * hangman dependencies leaked into this PR, removing them * add bs4 and lxml back to lock file * Capitalize comments Co-authored-by: Bluenix <[email protected]> * Improving comments (capitalization) Co-authored-by: Bluenix <[email protected]> * polishing * explaining that self.original_message will be set later in the callback function of the dropdown * fixing nitpicks * cast to integer from hex * removing unnecessary trailing commas * Simplifying L274-L276 Co-authored-by: Bluenix <[email protected]> * Add ellipsis to end of description if it's too long Co-authored-by: Bluenix <[email protected]> * Changing to hex Co-authored-by: Bluenix <[email protected]> * Running blocking function (BeautifulSoup.find_all) to thread Co-authored-by: Bluenix <[email protected]> * logger.error errors * Fixing error with to_thread * Fixing errors with MAPPING_OF_KYU Co-authored-by: Bluenix <[email protected]> * changing `query` to `-query` if the query is a kata level * changing embed names to add the kata name * Mimicking mailing list's behavior Co-authored-by: Bluenix <[email protected]> * url attribute for all embeds & title for all embeds * remove view after a certain amount of tikme * disabling view after waiting instead of just editing it out * styling * remove view to avoid spamming errors * changing `logger` to `log` Co-authored-by: Xithrius <[email protected]> * Change `logger` to `log` for logging errors Co-authored-by: ChrisJL <[email protected]> Co-authored-by: Bluenix <[email protected]> Co-authored-by: Xithrius <[email protected]>
* | | Merge pull request #904 from python-discord/update-advent-of-code-channel-idsGravatar Xithrius2021-10-13-2/+2
|\ \ \ | | | | | | | | update advent of code channel IDs
| * | | update advent of code channel IDsGravatar Chris Lovering2021-10-13-2/+2
|/ / / | | | | | | | | | We deleted and re-made the channels so new IDs are needed.
| * | Change pascal's triangle imageGravatar TizzySaurus2021-10-02-1/+1
| | |
| | * Address review & make `AI.get_move` a staticmethodGravatar Izan2021-10-17-5/+7
| | |
| | * Add missing `Game.channel` attributeGravatar Izan2021-10-11-0/+1
| | |
| | * Set `AI.user` to @Sir LancebotGravatar Izan2021-10-11-4/+5
| |/ |/|
* | Merge pull request #897 from python-discord/command-prefix-fixGravatar ChrisJL2021-10-09-5/+5
|\ \ | | | | | | Replace usage of bot.command_prefix with constants.Client.prefix
| * | Replace usage of bot.command_prefix with constants.Client.prefixGravatar wookie1842021-10-09-5/+5
|/ /
* | Merge pull request #881 from camcaswell/duckduckduckgooseGravatar ChrisJL2021-10-08-16/+26
|\ \ | | | | | | Duckduckduckgoose
| * \ Merge branch 'main' into duckduckduckgooseGravatar ChrisJL2021-10-08-89/+18036
| |\ \ | |/ / |/| |
* | | Merge pull request #895 from python-discord/topic-improvementsGravatar Xithrius2021-10-08-19/+33
|\ \ \ | | | | | | | | `.topic` command improvements
| * \ \ Merge branch 'main' into topic-improvementsGravatar Xithrius2021-10-08-1/+1
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #893 from ↵Gravatar ChrisJL2021-10-08-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/bug/candy-collection/fix-typeerror-id Candy collection: fix positional arg being passed as kword
| * | | | Candy collection: fix positional arg being passed as kwordGravatar Matteo Bertucci2021-10-08-1/+1
|/ / / / | | | | | | | | This caused a `TypeError` to be raised, as the `id` argument could only be used as a positional argument and not by keyword.
| * | | Remove unnecessary `else`Gravatar Izan2021-10-08-7/+7
| | | |
| * | | Add handling for `discord.NotFound` when re-rolling / removing reactionGravatar Izan2021-10-08-2/+7
| | | |
| * | | `.topic` command improvements.Gravatar Izan2021-10-08-18/+27
|/ / / | | | | | | | | | | | | | | | - Fix bug where command author couldn't re-roll - Now removes user's reaction up re-roll - Added a missing `break` statement
* | | Merge pull request #880 from python-discord/topic-command-cooldownGravatar Xithrius2021-10-07-17/+60
|\ \ \ | | | | | | | | Add a 2 minute cooldown to the topic command
| * | | Allow topics to be refreshedGravatar Chris Lovering2021-10-07-3/+62
| | | | | | | | | | | | | | | | | | | | | | | | This is done via an emoji as buttons are too big Co-authored-by: Bluenix <[email protected]>
| * | | Add a 2 minute cooldown to the topic commandGravatar Chris Lovering2021-10-07-20/+4
|/ / / | | | | | | | | | Using the command while it's on cooldown will hit the error handler, which sends an error message showing how long is left on the cooldown, which is deleted after 7.5 seconds.
* | | Added Anagrams command (#874)Gravatar Mayur Odedara2021-10-07-0/+17778
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added Anagrams command Added 2 files anagram.py - Has the code for anagram command anagram.json - Contains all the words for anagram command * Update bot/exts/fun/anagram.py Using "with" for resource file instead of getting data directly Co-authored-by: brad90four <[email protected]> * Update bot/exts/fun/anagram.py Updated title text for answer embed Co-authored-by: Bluenix <[email protected]> * Update bot/exts/fun/anagram.py Comma separated text for winners list Co-authored-by: Bluenix <[email protected]> * Updated anagram.py as per review comments -Removed redundant variables -Updated embed text to avoid 'all' -Updated stale comments * Some minor formatting fixes -Added trailing commas to embed -Updated all embeds to have consistent format * Polish anagram command for multiple channels * Updated docstrings * Allowed command to be used in multiple channels * Create a class for anagram game instances * Lay groundwork for threads Co-Authored-By: Bluenix <[email protected]> * Updated resource file for anagram command * Anagrams are now cross referenced with list of common words which should be easy for users to guess * It should not have any slur words * Update bot/exts/fun/anagram.py Co-authored-by: brad90four <[email protected]> * Update bot/exts/fun/anagram.py Co-authored-by: brad90four <[email protected]> * Update bot/exts/fun/anagram.py Co-authored-by: brad90four <[email protected]> * Update bot/exts/fun/anagram.py Co-authored-by: brad90four <[email protected]> * Linting error fix Linting error fix * Error fix Removed the "seconds" causing issue for anagram command * Revert "Error fix" This reverts commit 8c00d70f9faf62c536eac1fa61877dfab328a83f. * Error fix for seconds Fixed the error by removing "seconds" Co-authored-by: brad90four <[email protected]> Co-authored-by: Bluenix <[email protected]> Co-authored-by: Xithrius <[email protected]>
* | | Merge pull request #892 from python-discord/catch-403-from-Typing-callsGravatar Johannes Christ2021-10-06-55/+85
|\ \ \ | | | | | | | | Monkey patch http.send_typing to catch 403s
| * | | Monkey patch http.send_typing to catch 403sGravatar Chris Lovering2021-10-04-55/+85
|/ / / | | | | | | | | | Sometimes discord turns off typing events by throwing 403's, so we should catch those
* | | Merge pull request #890 from python-discord/better-bookmark-error-messageGravatar Matteo Bertucci2021-10-03-1/+7
|\ \ \ | | | | | | | | Give the bookmark command a better error message
| * \ \ Merge branch 'main' into better-bookmark-error-messageGravatar wookie1842021-10-03-5/+6
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #785 from PythonTryHard/mainGravatar wookie1842021-10-03-1/+2
|\ \ \ \ | | | | | | | | | | Correct an off-by-one bug due to bisect.bisect()
| * \ \ \ Merge branch 'main' into mainGravatar wookie1842021-10-03-15/+96
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #891 from python-discord/spooky-ignore-botsGravatar Matteo Bertucci2021-10-03-4/+4
|\ \ \ \ \ | | | | | | | | | | | | Ignore all bot messages for spooky react
| * | | | | Ignore bot messages for spooky reactGravatar Gustav Odinger2021-10-03-4/+4
|/ / / / / | | | | | | | | | | | | | | | | | | | | Previously only ignored its own messages, but now ignores messages from all bots.
| * | | | Remove trailing whitespaceGravatar PythonTryHard2021-09-24-1/+1
| | | | |
| * | | | Bisect only on love percent thresholdsGravatar PythonTryHard2021-09-24-1/+2
| | | | |
| | * | | Give the bookmark command a better error messageGravatar Chris Lovering2021-10-03-1/+7
| |/ / / |/| | |
* | | | Merge pull request #877 from python-discord/bug/875/emoji-offet-naiveGravatar ChrisJL2021-10-03-6/+6
|\ \ \ \ | |_|_|/ |/| | | Emoji: make the datetimes offset-naive
| * | | Merge branch 'main' into bug/875/emoji-offet-naiveGravatar wookie1842021-10-03-5/+86
| |\ \ \ | |/ / / |/| | |
* | | | `.quack` (#849)Gravatar Kronifer2021-10-01-0/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * feat: Added quack command * added log.error call for request fails * spacing change Co-authored-by: Bluenix <[email protected]> * another spacing change Co-authored-by: Bluenix <[email protected]> * Moved description to footer Co-authored-by: Bluenix <[email protected]> * whitespace fix * chore: Removed the link from the footer and set it as the url param * chore: moved footer to description Co-authored-by: Bluenix <[email protected]> Co-authored-by: Xithrius <[email protected]>
* | | | Allow everyone to use the `.bm` command everywhere (#885)Gravatar Objectivitix2021-09-26-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Allow everyone to use the bm command * Add everyone role in Roles constants * Use envvars and re-order Roles section to be more organized * Fix trailing whitespace We might need to squash merge, four commits for a single small fix is too much
| * | | Hackto issue finder: make d.py timestamp naiveGravatar Matteo Bertucci2021-10-01-2/+2
| | | |
| * | | Hacktoberfest: make datetimes offset-naiveGravatar Matteo Bertucci2021-09-21-2/+2
| | | |
| * | | Emoji: make the datetimes offset-naiveGravatar Matteo Bertucci2021-09-20-2/+2
| |/ / | | | | | | | | | | | | You know the drill, due to discord.py 2.0a0 datetimes are now offset-aware, breaking some code. Closes python-discord/sir-lancebot#875
| | * Make setting the old embed description cleanerGravatar camcaswell2021-10-02-3/+1
| | |