aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar Steele2020-11-26-1/+1
|\ | | | | | | Stelercus/close_command
| * Merge pull request #1307 from ↵Gravatar Sebastiaan Zeeff2020-11-26-1/+1
| |\ | | | | | | | | | | | | python-discord/sebastiaan/bugs/add-voiceban-to-scheduler-types Add voice_ban to supported types of the infraction scheduler
| | * Add voice_ban to supported types of the schedulerGravatar Sebastiaan Zeeff2020-11-26-1/+1
| |/ | | | | | | | | | | The `voice_ban` infraction was not listed as a supported type for the infraction scheduler. This meant that the scheduler did not schedule the expiry of `voice_ban` infractions after a restart. Those unlucky users were voice-banned perpetually.
* | Changes requested by @MarkKoz, new `unclaim_channel` methodGravatar Steele2020-11-26-20/+26
| | | | | | | | Deleted expensive logging operation; moved cooldown role removal functionality to new `unclaim_channel` method; handle possibility that claimant has left the guild; optimized redis cache iteration with `any`
* | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar Steele2020-11-25-109/+167
|\| | | | | | | Stelercus/close_command
| * Add Mark as a code owner of CI and Docker filesGravatar Mark2020-11-24-0/+5
| |
| * Specify code ownership for MarkGravatar MarkKoz2020-11-24-0/+19
| |
| * Update CODEOWNERSGravatar Dennis Pham2020-11-24-2/+2
| |
| * Update CODEOWNERSGravatar Joe Banks2020-11-24-1/+2
| |
| * Update CODEOWNERSGravatar Joe Banks2020-11-24-1/+1
| |
| * Open deployment.yaml from kubernetes repositoryGravatar Sebastiaan Zeeff2020-11-23-22/+4
| | | | | | | | | | | | | | | | We will now use the deployment information located in the private python-discord/kubernetes repository. The workflow will use a GitHub Personal Access Token to access this private repository. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| * Narrow down repository events that trigger a buildGravatar Sebastiaan Zeeff2020-11-23-1/+1
| | | | | | | | | | | | | | | | I've narrowed down repository events that trigger a Build to the "push" event specifically. This means that we never build for a "pull request" trigger, even if the source branch is called "master". Signed-off-by: Sebastiaan Zeeff <[email protected]>
| * 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
| | |\ \ \ | | |/ / / | |/| | |
| | * | | Cover Unicode emojis catching as antispam rule with test casesGravatar ks1292020-11-20-4/+25
| | | | |
| | * | | Catch Unicode emojis in emojis filtering ruleGravatar ks1292020-11-20-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | Instead detecting only custom emojis, rule now catch too Unicode emojis. This converts Unicode emojis to :emoji: format and count them.
| | * | | Install emoji package for emojis filteringGravatar ks1292020-11-20-19/+31
| | | | |
| | | * | Remove duplicate checkmarkGravatar Gustav Odinger2020-11-21-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - This removes a duplicate checkmark from the title of the embed - The checkmark was left from the previous title system and wasn't removed in the change
| | | * | Update available help channel embed messageGravatar Gustav Odinger2020-11-21-1/+1
| | | | | | | | | | | | | | | | | | | | - This changes "we'll try to help you" to say "others will try to help you" - Clarifies that the rest of the community is going to help — not some dedicated help/staff team
| | | * | Update available message to sound betterGravatar Gustav Odinger2020-11-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | - This replaces "one" with "any" - This is supposed to read better
| | | * | Use author as the title of the embedGravatar Gustav Odinger2020-11-17-1/+1
| | | | | | | | | | | | | | | | | | | | - Allows the icon to be centered
| | | * | Add green-checkmark to bot constantsGravatar Gustav Odinger2020-11-17-0/+4
| | | | |
| | | * | Update help channel available messageGravatar Gustav Odinger2020-11-16-10/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Adds a footer and title - Uses a green colored embed - Updates message to be easier to read and contain practical info for asking better questions
| | | * | Add bright green color to constantsGravatar Gustav Odinger2020-11-16-0/+2
| | |/ / | | | | | | | | | | | | - The color is used in the new help channel embed
| | | * Make `additional_info` non-optional.Gravatar kosayoda2020-10-20-1/+1
| | | | | | | | | | | | | | | | | | | | The `Optional` typehint suggests allowing None as a value, which does not make sense as a message in the logs.
| | | * Improve default argument.Gravatar kosayoda2020-10-08-4/+1
| | | |
| | | * Resolve logic error with reason override.Gravatar kosayoda2020-10-08-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The reason override for the user message should be set to the infraction reason *if* the override is None, not if it isn't. The parameter is also renamed to `user_reason` for better clarity.
| | | * 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.
* | | | Moved the removal of the cooldown role from `close_command` to `move_to_dormant`Gravatar Steele2020-11-25-7/+8
| | | |
* | | | `!close` removes role when they have no help channels left; needs to be ↵Gravatar Steele2020-11-25-1/+6
| | | | | | | | | | | | | | | | fixed so role is removed when the channel times out
* | | | `!close` removes the cooldown role from the claimant even when invoked by ↵Gravatar Steele2020-11-20-10/+15
|/ / / | | | | | | | | | someone else; flattened `close_command`
* | | 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.
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-31 15:05:43 +0000