aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | | | | | | | | | | | | | | | | | | Modify `!superstar` to use `apply_infraction`.Gravatar kosayoda2020-10-01-49/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using `apply_infraction` from `InfractionScheduler` rather than doing it manually allows us to handle HTTP errors while reducing code duplication. Specifically, discord.Forbidden is handled when the bot tries to superstar someone they do not have permissions to. Resolves BOT-5Q.
| | | | | * | | | | | | | | | | | | | | | | | fix(bot): PR reivew of bot.pyGravatar Thomas Petersson2020-11-27-8/+10
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | * | | | | | | | | | | | | | | | | | Merge branch 'master' into fix/tagp/graceful-stasd-connectGravatar Thomas Petersson2020-11-09-907/+2421
| | | | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | |/ / / / / / / / / / / / / / / / / / | | | | |/| | | | | | | | | | | | | | | | | |
| | | | | * | | | | | | | | | | | | | | | | | fix(bot): statds pr review suggestionsGravatar Thomas Petersson2020-11-09-6/+14
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | * | | | | | | | | | | | | | | | | | fix(bot): better fallback logicGravatar Thomas Petersson2020-10-05-3/+3
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | * | | | | | | | | | | | | | | | | | fix(bot): refactor of connect_statsdGravatar Thomas Petersson2020-10-05-9/+5
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | * | | | | | | | | | | | | | | | | | fix(bot): redundant false checks on self.statsGravatar Thomas Petersson2020-10-05-11/+3
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | * | | | | | | | | | | | | | | | | | fix(bot): Not assign stats to NoneGravatar Thomas Petersson2020-10-05-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | self.stats is referred to as bot.stats in the project, which was overlooked. This should "disable" stats until it's successfully reconnected. The retry attempts will continue until it stops throwing or fails 10x
| | | | | * | | | | | | | | | | | | | | | | | fix(statsd): Gracefully handle gaierroGravatar Thomas Petersson2020-10-05-4/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per issue #1185 the bot might go down if the statsd client fails to connect during instantiation. This can be caused by an outage on their part, or network issues. If this happens getaddrinfo will raise a gaierror. This PR catched the error, sets self.stats to None for the time being, and handles that elsewhere. In addition a fallback logic was added to attempt to reconnect, in the off-chance it's a temporary outage
| | | | | | | * | | | | | | | | | | | | | | | Added docstring for the user activity function.Gravatar Xithrius2020-12-10-1/+6
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | | | | | | | | Merge branch 'moderation/user-from-metricity' of ↵Gravatar Xithrius2020-12-10-12/+7
| | | | | | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot into moderation/user-from-metricity
| | | | | | | | * | | | | | | | | | | | | | | | Slightly reformatted activity block building.Gravatar Xithrius2020-12-10-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Hassan Abouelela <[email protected]>
| | | | | | | | * | | | | | | | | | | | | | | | Compressed embed buildingGravatar Xithrius2020-12-10-11/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Hassan Abouelela <[email protected]>
| | | | | | | * | | | | | | | | | | | | | | | | Compressed if into or statements.Gravatar Xithrius2020-12-10-9/+2
| | | | | | | |/ / / / / / / / / / / / / / / /
| | | | | | | * | | | | | | | | | | | | | | | Changed logic of membership info creation.Gravatar Xithrius2020-11-11-14/+20
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | | | | | | | | Moved activity data further up in embed.Gravatar Xithrius2020-11-06-14/+31
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | | | | | | | | User command gets verification time and message count.Gravatar Xithrius2020-11-04-1/+21
| | | | | | | | | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | | | | | | | | Remove selenium from the element listGravatar Matteo Bertucci2020-11-12-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This could lead to some confusion with the users believing that this channel is reserved to help related to the selenium tool.
* | | | | | | | | | | | | | | | | | | | | | | Merge branch 'master' into error-handler-testGravatar ks1292020-11-21-2526/+3832
|\| | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | Merge pull request #1287 from python-discord/help-channel-msgGravatar Boris Muratov2020-11-21-10/+25
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated available help channel embed
| | * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into help-channel-msgGravatar Boris Muratov2020-11-21-371/+514
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1293 from ks129/emojis-filterGravatar Daniel Brown2020-11-20-26/+61
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|/ / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | Include Unicode emojis to emojis filter
| | * | | | | | | | | | | | | | | | | | | | | | Merge branch 'master' into emojis-filterGravatar Daniel Brown2020-11-20-345/+453
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | Checkout code so we can deployGravatar Sebastiaan Zeeff2020-11-19-0/+9
| | | | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1291 from python-discord/sebastiaan/backend/fix-deploy-stageGravatar Sebastiaan Zeeff2020-11-19-172/+36
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the deploy stage of our build pipeline
| | * | | | | | | | | | | | | | | | | | | | | | | Fix the deploy stage of our build pipelineGravatar Sebastiaan Zeeff2020-11-19-172/+36
| |/ / / / / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've fixed the deploy stage of our build pipeline, as it got mixed in with the old workflow file due to a merge conflict. The deploy stage is currently split into a separate workflow; theoretically, this allows us to trigger a redeploy from GitHub, without having to build the container image again. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| * | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1286 from python-discord/kubernetes-deployGravatar Joe Banks2020-11-19-3/+195
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| | * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into kubernetes-deployGravatar Leon Sandøy2020-11-19-286/+486
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1290 from ↵Gravatar Sebastiaan Zeeff2020-11-19-59/+80
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/sebastiaan/backend/improve-actions-workflow Ensure flake8 actually lints pull requests
| | * | | | | | | | | | | | | | | | | | | | | | | | Add comment explaining buildx to workflowGravatar Sebastiaan Zeeff2020-11-19-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's better to document these steps. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | | | | | | | | | | | | | | | | | | | | Use GHCR image tags in PipfileGravatar Sebastiaan Zeeff2020-11-18-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | | | | | | | | | | | | | | | | | | | | Update badges in README to new workflowsGravatar Sebastiaan Zeeff2020-11-18-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | | | | | | | | | | | | | | | | | | | | Make sure we lint the actual pull requestGravatar Sebastiaan Zeeff2020-11-18-55/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unfortunately, our old setup did not actually lint the PR, as it was running in the context of the target repository. To sidestep the issue of using `pull_request_target` altogether, I've now changed our run of flake8 to using it directly and having it output its errors in teh format of Workflow Commands. This means that our flake8 output will not be translated automatically in annotations for the run. In addition, I've split up the workflow into two separate files: one for linting & testing and one for building (& deploying). Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | | | | | | | | | | | | | | | | | | | | Pull snekbox image from GHCR in docker-composeGravatar Sebastiaan Zeeff2020-11-18-1/+1
| |/ / / / / / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're in the process of migrating snekbox to the GitHub Container Repository, which will replace DockerHub. I've changed docker-compose to reflect that change. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| * | | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1245 - global bot instanceGravatar Mark2020-11-16-282/+304
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Merge master and fix conflictsGravatar MarkKoz2020-11-16-747/+1885
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Set up Sentry when running rather than upon importGravatar MarkKoz2020-10-17-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was causing an error if a DSN was not configured. It also feels wrong and confusing to attempt to make a connection just upon import.
| | * | | | | | | | | | | | | | | | | | | | | | | Move bot creation code from __main__.py to bot.pyGravatar MarkKoz2020-10-17-53/+58
| | | | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Move logging set up to a separate moduleGravatar MarkKoz2020-10-17-83/+93
| | | | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Syncers: make functions staticGravatar MarkKoz2020-10-17-65/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The classes no longer hold any state since they can use the global bot instance.
| | * | | | | | | | | | | | | | | | | | | | | | | Use global bot instance in syncersGravatar MarkKoz2020-10-16-45/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They're pretty close to being fully static classes, but it's difficult to make the name attribute a static abstract property.
| | * | | | | | | | | | | | | | | | | | | | | | | Use global bot instance in InterpreterGravatar MarkKoz2020-10-16-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Use global bot instance in try_get_channelGravatar MarkKoz2020-10-16-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Use global bot instance in wait_for_deletionGravatar MarkKoz2020-10-16-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Use global bot instance in send_to_paste_serviceGravatar MarkKoz2020-10-16-27/+28
| | | | | | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | | | | | Add global bot instanceGravatar MarkKoz2020-10-16-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Is **very** convenient when writing utility functions that rely on the bot's state, but aren't in cogs and therefore lack the typical way to access the instance. No more passing around of the instance as an arg!
| | | * | | | | | | | | | | | | | | | | | | | | | Update snekbox address in config-default.ymlGravatar Joe Banks2020-11-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | | | Update config options with new hostsGravatar Joe Banks2020-11-15-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | | | Add deploy steps to GitHub ActionsGravatar Joe Banks2020-11-15-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | | | Add Kubernetes deployment manifestGravatar Joe Banks2020-11-15-0/+21
| | |/ / / / / / / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | |