aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Role lookup takes place only in `_handle_role_change`.Gravatar swfarnsworth2021-05-31-7/+4
|
* Merge branch 'main' of https://github.com/python-discord/bot into ↵Gravatar swfarnsworth2021-05-31-16/+21
|\ | | | | | | swfarnsworth/one_help_channel
| * Merge pull request #1606 from Numerlor/doc-force-prefix-priorityGravatar Xithrius2021-05-30-16/+21
| |\ | | | | | | Improve doc symbol name conflict resolution
| | * Merge branch 'main' into doc-force-prefix-priorityGravatar Xithrius2021-05-30-4/+113
| | |\ | | |/ | |/|
| | * Add the doc group to FORCE_PREFIX_GROUPSGravatar Numerlor2021-05-21-0/+1
| | | | | | | | | | | | | | | | | | Symbols with the doc group refer to the pages themselves without pointing at a specific element in the HTML. In most cases we can't properly parse those so this change will move them out of the way for other symbols to take priority.
| | * Prioritize symbols depending on their group's pos in FORCE_PREFIX_GROUPSGravatar Numerlor2021-05-21-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When both symbols are in FORCE_PREFIX_GROUPS, instead of relying on which symbol comes later and always renaming the latter one, the symbols with a lower priority group are moved out of the way instead of forcing the new symbol to be moved. This will help make relevant symbols be more likely to come up when searching the docs. The constant was reordered by the priority of the groups to work with this change
| | * Prefer using the package name as a prefix when handling symbol conflictsGravatar Numerlor2021-05-21-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | When renaming, having label.symbol etc. in the list of the renamed symbol in the embed doesn't really provide much context on what that symbol may be to the user. Prioritizing the package name instead should makes it clearer that it's from an another package. Conflicts within a package are still resolved using the previous logic
* | | Re-introduced static method for role change exception handling.Gravatar swfarnsworth2021-05-29-2/+21
| | | | | | | | | | | | A function that did the same thing previously existed in `_cooldown.py`.
* | | Merge branch 'main' of https://github.com/python-discord/bot into ↵Gravatar swfarnsworth2021-05-29-3/+112
|\| | | | | | | | | | | swfarnsworth/one_help_channel
| * | Merge pull request #1615 from python-discord/reviews-revert-orderGravatar ChrisJL2021-05-27-1/+2
| |\ \ | | | | | | | | Recruitment: reverse nomination order
| | * | Recruitment: reverse nomination orderGravatar Matteo Bertucci2021-05-27-1/+2
| |/ / | | | | | | | | | This makes review appear in chronological order, making it easier to say things like "^^ what they said".
| * | Merge pull request #1614 from python-discord/update-pixels-regexGravatar Boris Muratov2021-05-25-1/+1
| |\ \ | | | | | | | | Update pixels token regex to false match less
| | * | Update pixels token regex to false match lessGravatar Chris2021-05-25-1/+1
| |/ /
| * | Merge pull request #1613 from ChrisLovering/pixels-token-filteringGravatar Joe Banks2021-05-24-0/+108
| |\ \ | | | | | | | | Add filter for pixels API tokens
| | * | Add filter for pixels API tokensGravatar Chris2021-05-24-0/+108
| |/ /
| * | Merge pull request #1612 from python-discord/higher-duck-limitGravatar Boris Muratov2021-05-24-1/+1
| |\ \ | | | | | | | | Up duck pond threshold to 7
| | * | Up duck pond threshold to 7Gravatar Gustav Odinger2021-05-24-1/+1
| |/ / | | | | | | | | | | | | Makes duck pond entries less common, by requiring more ducks for a message to be ducked
| * | Merge pull request #1610 from python-discord/fix/float-tag-typoGravatar ChrisJL2021-05-23-1/+1
| |\ \ | | | | | | | | Remove the extra 'as' in the floats tag
| | * | fix: Remove the extra 'as' in floats.mdGravatar ToxicKidz2021-05-23-1/+1
| |/ /
* | | Remove `claim_minutes` constant.Gravatar swfarnsworth2021-05-21-1/+0
| | |
* | | Merge branch 'main' of https://github.com/python-discord/bot into ↵Gravatar swfarnsworth2021-05-21-1108/+2045
|\| | | | | | | | | | | swfarnsworth/one_help_channel
| * | Merge pull request #1607 from python-discord/vcokltfre/chore/modmail-tag-updateGravatar ChrisJL2021-05-21-1/+1
| |\ \ | | |/ | |/| chore: use new moderators role in modmail tag
| | * chore: use new moderators roleGravatar vcokltfre2021-05-21-1/+1
| |/
| * Merge pull request #1593 from python-discord/flake-8-isn't-a-taskGravatar Hassan Abouelela2021-05-17-1/+1
| |\ | | | | | | Fix pre-commit, since flake8 isn't a task
| | * Fix pre-commit, since flake8 isn't a taskGravatar Chris2021-05-17-1/+1
| |/
| * Merge pull request #1558 from doublevcodes/mainGravatar ChrisJL2021-05-16-0/+28
| |\ | | | | | | Added blocking tag
| | * Merge branch 'main' into mainGravatar ChrisJL2021-05-16-1549/+2249
| | |\ | | |/ | |/|
| * | Merge pull request #1590 from python-discord/incident-emojisGravatar ChrisJL2021-05-16-3/+3
| |\ \ | | | | | | | | Constants: use in-server emojis for incident
| | * | Constants: use in-server emojis for incidentGravatar Matteo Bertucci2021-05-16-3/+3
| |/ / | | | | | | The default config is currently referencing the emoji server versions, making it unable to work with the nomination archive automation
| * | Merge pull request #1549 from python-discord/nomination-archive-automationGravatar ChrisJL2021-05-16-20/+164
| |\ \ | | | | | | | | Nomination archive automation
| | * \ Merge branch 'main' into nomination-archive-automationGravatar ChrisJL2021-05-16-1537/+2963
| | |\ \ | | |/ / | |/| |
| * | | Merge pull request #1589 from python-discord/fix-dockerfileGravatar ChrisJL2021-05-16-1/+1
| |\ \ \ | | | | | | | | | | Hotfix: Fixes Dependency Install Flag In Dockerfile
| | * | | Fixes Dependency Install Flag In DockerfileGravatar Hassan Abouelela2021-05-16-1/+1
| |/ / / | | | | | | | | | | | | Signed-off-by: Hassan Abouelela <[email protected]>
| * | | Merge pull request #1582 from python-discord/39-poetryGravatar Hassan Abouelela2021-05-16-1084/+1650
| |\ \ \ | | | | | | | | | | Upgrade To 3.9 and Poetry
| | * \ \ Merge branch 'main' into 39-poetryGravatar Hassan Abouelela2021-05-16-1/+200
| | |\ \ \ | | |/ / / | |/| | |
| * | | | Merge pull request #1573 from python-discord/extract-metabase-dataGravatar Joe Banks2021-05-13-0/+198
| |\ \ \ \
| | * | | | Delete reddit cog constantsGravatar Chris2021-05-13-15/+0
| | | | | | | | | | | | | | | | | | | | | | | | These were added back accidentaly as part of a merge conflict.
| | * | | | Merge branch 'main' into extract-metabase-dataGravatar Joe Banks2021-05-13-2/+26
| | |\ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge pull request #1584 from Numerlor/docker-overrideGravatar Dennis Pham2021-05-12-0/+1
| |\ \ \ \ \ | | | | | | | | | | | | | | Add docker-compose.override.yml to gitignore
| | * \ \ \ \ Merge branch 'main' into docker-overrideGravatar Dennis Pham2021-05-12-2/+25
| | |\ \ \ \ \ | | |/ / / / / | |/| | | | |
| | * | | | | Add docker-compose.override.yml to gitignoreGravatar Numerlor2021-05-12-0/+1
| | | | | | |
| | | * | | | Merge branch 'main' into extract-metabase-dataGravatar ChrisJL2021-05-09-381/+16
| | | |\ \ \ \ | | | |/ / / / | | |/| | | |
| | | * | | | Move a long comment in Metabse cog into the func doc stringGravatar Chris2021-05-09-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Xithrius <[email protected]>
| | | * | | | Type hint the Metabase cog!Gravatar Chris2021-05-09-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Xithrius <[email protected]>
| | | * | | | Revert changes to int eGravatar Chris2021-05-09-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit reverts the changes to int e, by giving the invoker instructions on how to access the export via int e. This means the metabase exports are not inserted to every int e envrionment. I have also added a seperate message in this commit to handle when the paste service is offline.
| | | * | | | Pass raise_for_status as a kwarg for better readibilityGravatar Chris2021-05-09-28/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This means the handling of the response comes directly after the context manager rather than having to handle errors.
| | | * | | | Use allowed strings converter in Metabase cogGravatar Chris2021-05-09-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the need to manually validate user input. Co-authored-by: Numerlor <[email protected]>
| | | * | | | Remove unneeded context manager in Metabase cog Gravatar ChrisJL2021-05-09-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | StringIO as it has no underlying connections, so a context manager is not needed Co-authored-by: Numerlor <[email protected]>
| | | * | | | Update warn to warning, due to deprecation Gravatar ChrisJL2021-05-09-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit also includes a minor docstring change Co-authored-by: Numerlor <[email protected]>
| | | * | | | Fix minor grammer issues with metabase commentsGravatar Chris2021-05-09-4/+5
| | | | | | |