aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge pull request #1475 from python-discord/mbaruh/review_fixGravatar Joe Banks2021-03-19-5/+8
|\ | | | | Fix review formatting when there's only one infraction type
| * Fix review formatting when there's only one infraction typeGravatar Boris Muratov2021-03-19-5/+8
|/
* Merge pull request #1459 from python-discord/mbaruh/autoreviewGravatar Joe Banks2021-03-19-31/+467
|\ | | | | Talentpool Autoreview
| * Merge branch 'mbaruh/autoreview' of https://github.com/python-discord/bot ↵Gravatar Boris Muratov2021-03-19-229/+174
| |\ | | | | | | | | | into mbaruh/autoreview
| | * Merge branch 'main' into mbaruh/autoreviewGravatar Joe Banks2021-03-19-229/+174
| | |\ | |_|/ |/| |
* | | Merge pull request #1435 from python-discord/swfarnsworth/infraction_messageGravatar Matteo Bertucci2021-03-17-26/+32
|\ \ \ | | | | | | | | Instructions to dispute an infraction vary by infraction type.
| * \ \ Merge branch 'main' into swfarnsworth/infraction_messageGravatar Matteo Bertucci2021-03-17-429/+358
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1467 from ↵Gravatar Kieran Siek2021-03-17-0/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/bug/info/bot-xr/code-block-null-byte Code block: remove null bytes before parsing AST
| * \ \ \ Merge branch 'main' into bug/info/bot-xr/code-block-null-byteGravatar Kieran Siek2021-03-17-3/+4
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #1468 from ↵Gravatar Matteo Bertucci2021-03-16-3/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/info/bot-xk/defcon-threshold-none Account for defcon threshold being None when displaying server info
| * | | | | Info: account for defcon threshold being NoneGravatar MarkKoz2021-03-15-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | Fixes BOT-XK
| * | | | | Defcon: fix naming conflict between threshold cmd and attributeGravatar MarkKoz2021-03-15-2/+2
|/ / / / /
| * / / / Code block: remove null bytes before parsing ASTGravatar MarkKoz2021-03-15-0/+3
|/ / / / | | | | | | | | | | | | | | | | | | | | `ast.parse` raises a ValueError complaining that source code strings cannot contain null bytes. It seems like they may accidentally get pasted into Discord by users sometimes.
* | | | Merge pull request #1465 from python-discord/gitattributesGravatar Dennis Pham2021-03-14-2/+1
|\ \ \ \ | | | | | | | | | | Use .gitattributes to normalise line endings on check-in
| * | | | Use .gitattributes to normalise line endings on check-inGravatar MarkKoz2021-03-13-2/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the mixed line endings pre-commit hook because it is obsolete. Relying on git to handle line endings means contributors have more flexibility with which line endings they want to use on check-out. The settings in .gitattributes only impose which line endings will be used upon check-in (LF), which should not impact local development; git will still respect the core.eol and core.autocrlf settings.
* | | | master => mainGravatar Joe Banks2021-03-13-22/+22
| | | |
* | | | Update help channel names from chemical elements to fruitGravatar Joe Banks2021-03-13-126/+59
| | | | | | | | | | | | | | | | | | | | * Update and rename elements.json to fruits.json * Update _name.py * Update _cog.py
* | | | Merge pull request #1461 from python-discord/simplify-pypiGravatar ks1292021-03-13-21/+12
|\ \ \ \ | | | | | | | | | | Simplifying the PyPi command.
| * | | | Added '_' to allowed chars, shortened embed.Gravatar xithrius2021-03-12-21/+12
|/ / / /
* | | | Merge pull request #1455 from python-discord/many-snowflakesGravatar Matteo Bertucci2021-03-12-11/+16
|\ \ \ \ | | | | | | | | | | Allow the sf command to take more than one snowflake at the time
| * \ \ \ Merge branch 'master' into many-snowflakesGravatar Xithrius2021-03-12-262/+248
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Fix typo in the token removerGravatar Matteo Bertucci2021-03-12-1/+1
| | | | |
* | | | | Merge pull request #1452 from python-discord/pypi-delete-on-not-foundGravatar Matteo Bertucci2021-03-12-27/+35
|\ \ \ \ \ | | | | | | | | | | | | Remove invoked command and message after failure.
| * \ \ \ \ Merge branch 'master' into pypi-delete-on-not-foundGravatar Matteo Bertucci2021-03-12-3/+2
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge PR #1460 - compose: read all env vars from '.env'Gravatar Mark2021-03-11-3/+2
|\ \ \ \ \ \
| * | | | | | Compose: read all environment variables from '.env'Gravatar kwzrd2021-03-12-4/+2
| | | | | | |
| * | | | | | Compose: read GitHub API key from '.env'Gravatar kwzrd2021-03-11-0/+1
| | | | | | |
| | * | | | | Remove invoked command and message after failure.Gravatar xithrius2021-03-11-27/+35
| |/ / / / / |/| | | | |
| | * | | | Restrict non-staffer to one snowflake at the timeGravatar Matteo Bertucci2021-03-08-1/+5
| | | | | |
| | * | | | Make the snowflake command accept many snowflakesGravatar Matteo Bertucci2021-03-08-12/+13
| | | | | |
| | | * | | Merge branch 'master' into swfarnsworth/infraction_messageGravatar Steele Farnsworth2021-03-06-452/+723
| | | |\ \ \ | | |_|/ / / | |/| | | |
| | | * | | Made multiline concatenated string conform to a certain style. Gravatar Steele Farnsworth2021-03-06-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | That style is not currently enforced by the linter. Co-authored-by: Matteo Bertucci <[email protected]>
| | | * | | Update the tests to reflect changes in expected behavior.Gravatar swfarnsworth2021-02-28-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | The DM sent to infracted users now instructs them to DM modmail if they want to discuss non-ban infractions, so the tests now check if that instruction is present. Note that there already exists a superfluous test for note infractions, for which no DM is sent by design.
| | | * | | Instructions to dispute an infraction vary by infraction type.Gravatar swfarnsworth2021-02-25-24/+26
| | | | | | | | | | | | | | | | | | | | | | | | Previously, the user was instructed to email the appeals email for infraction types that don't remove one from the server. They are now instructed to DM ModMail except for Ban-type infractions. Also removed the URL string literal from the hyperlink to that URL.
| | | | * | Review commands now use the user ID instead of nomination IDGravatar Boris Muratov2021-03-19-27/+19
| | | | |/ | | | | | | | | | | | | | | | The user ID is much more accessible, and is usually what is used to obtain the nomination ID.
| | | | * Don't reschedule reviews that are long overdueGravatar Boris Muratov2021-03-19-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If it's been over a day overdue for a review, don't reschedule it. This is done in order to not fire reviews for all nominations which are over 30 days old when the auto-reviewing feature is merged.
| | | | * Mark as reviewed when nominee is off serverGravatar Boris Muratov2021-03-19-3/+4
| | | | | | | | | | | | | | | | | | | | This is necessary as otherwise the bot would try to review them every time it restarts
| | | | * Default message in review when no nomination reason givenGravatar Boris Muratov2021-03-19-1/+1
| | | | |
| | | | * Replace mentions for ID's in watchlist listsGravatar Boris Muratov2021-03-19-1/+1
| | | | | | | | | | | | | | | | | | | | Uncached mentions render as 'invalid' users on mobile, and with the list now showing the user's name we can now just show the ID without many problems.
| | | | * Properly await coroutine in post_reviewGravatar Boris Muratov2021-03-19-1/+3
| | | | |
| | | | * Use ctx.send instead of ctx.channel.sendGravatar Boris Muratov2021-03-19-1/+1
| | | | | | | | | | | | | | | Co-authored-by: ToxicKidz <[email protected]>
| | | | * Add additional logging to _review.pyGravatar Boris Muratov2021-03-19-0/+8
| | | | |
| | | | * Apply requested style and grammar changesGravatar Boris Muratov2021-03-19-2/+2
| | | | |
| | | | * Apply requested grammar and style changes.Gravatar Boris Muratov2021-03-13-8/+13
| | | | |
| | | | * Use log instead of erroneous self.logGravatar Boris Muratov2021-03-13-1/+1
| | | | |
| | | | * Improve string building for long linesGravatar Boris Muratov2021-03-12-9/+15
| | | | |
| | | | * Reviewer no longer subclasses SchedulerGravatar Boris Muratov2021-03-12-5/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It didn't make much sense for the Reviewer to subclasses Scheduler. The Scheduler has methods that don't make sense to use on the Reviewer directly. There is now a Scheduler object as an attribute of the Reviewer. Interacting with it is done by adding __contains__, cancel, and cancel_all methods.
| | | | * The 'seen vote' emoji is now a random ducky.Gravatar Boris Muratov2021-03-11-4/+15
| | | | |
| | | | * Limit new commands to mods+Gravatar Boris Muratov2021-03-11-0/+2
| | | | |
| | | | * Added an auto-reviewer to the talentpool cogGravatar Boris Muratov2021-03-11-26/+404
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds the functionality to automatically review a nominee a set number of days after being nominated. This is implemented by subclassing the Scheduler and formatting a review after 30 days. The review contains details of the nominee, their nominations, the number of messages they have and the channels they're most active in, and statistics about their infractions and previous nominations. Lastly, the bot will add three emojis to the review: eyes to mark as seen, a thumbsup, and thumbsdown for the vote itself. The code accounts for the possibility of the review being too long for a single message but splitting it where necessary.
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-27 14:31:27 +0000