aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Refractors SendMessage FunctionGravatar Hassan Abouelela2020-11-24-39/+46
| | | | | | | Refractors the send message function in silence to make it more understandable and flexible. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Typo in DocGravatar Hassan Abouelela2020-11-24-1/+1
| | | Co-authored-by: Mark <[email protected]>
* Removes AnyChannel ConverterGravatar Hassan Abouelela2020-11-24-91/+3
| | | | | | | Removes the AnyChannel converter in favor of a combination of Text and Voice converters. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Voice Channel AccessGravatar Hassan Abouelela2020-11-24-1/+1
| | | | | | | A typo caused the function to return the text channel for `code/help 1`, when it is meant to access `code/help 2`. Signed-off-by: Hassan Abouelela <[email protected]>
* Refractors For Style GuidelinesGravatar Hassan Abouelela2020-11-24-13/+20
| | | | | | | Refractors method signatures and calls to follow python-discord style guide. Signed-off-by: Hassan Abouelela <[email protected]>
* Finalizes Silence & Unsilence UnitTestsGravatar Hassan Abouelela2020-11-23-17/+32
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Adds Silence & Unsilence UnitTestsGravatar Hassan Abouelela2020-11-23-1/+211
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Removes Redundant Exception HandlingGravatar Hassan Abouelela2020-11-23-38/+20
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* General Silence Class TestsGravatar Hassan Abouelela2020-11-23-17/+126
| | | | | | Adds tests for helper functions in the silence cog. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Voice Silence ReportingGravatar Hassan Abouelela2020-11-23-6/+5
| | | | | | | Fixes the channel reported as muted to voice channel chat channels when silencing voice channels. Signed-off-by: Hassan Abouelela <[email protected]>
* Fix Failing FunctionsGravatar Hassan Abouelela2020-11-23-1/+1
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Make Voice Channel Kick OptionalGravatar Hassan Abouelela2020-11-23-16/+34
| | | | | | | | | | Adds an optional parameter to the silence command to enable moderators to choose if they only update permissions, or kick members too. As an accompanying feature, the unsilence command now syncs voice channel permissions too. Signed-off-by: Hassan Abouelela <[email protected]>
* Merge branch 'voicechannel-mute' of https://github.com/HassanAbouelela/bot ↵Gravatar Hassan Abouelela2020-11-23-0/+0
|\ | | | | | | into voicechannel-mute
| * Write AnyChannelConverter TestsGravatar Hassan Abouelela2020-11-23-0/+48
| | | | | | | | Signed-off-by: Hassan Abouelela <[email protected]>
* | Write AnyChannelConverter TestsGravatar Hassan Abouelela2020-11-23-0/+48
|/ | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Add VC Mute Failure NotificationGravatar Hassan Abouelela2020-11-23-21/+24
| | | | | | | Notifies invocation channel that the silence command failed to silence the channel because it could not move members, but roles were updated. Signed-off-by: Hassan Abouelela <[email protected]>
* Add VC Mute FunctionalityGravatar Hassan Abouelela2020-11-22-2/+64
| | | | | | | Adds and calls a function to force a voice channel member to sync permissions. See #1160 for why this is necessary. Signed-off-by: Hassan Abouelela <[email protected]>
* Merge branch 'master' into voicechannel-muteGravatar Hassan Abouelela2020-11-22-430/+589
|\
| * Merge pull request #1204 from python-discord/superstar-fixGravatar Boris Muratov2020-11-21-49/+50
| |\ | | | | | | Modify `!superstar` to use `apply_infraction`.
| | * Empty commitGravatar mbaruh2020-11-21-0/+0
| | |
| | * Merge branch 'master' into superstar-fixGravatar Boris Muratov2020-11-21-1982/+3751
| | |\ | | |/ | |/|
| * | 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
| | | | | | |