aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | * | | | Add deploy steps to GitHub ActionsGravatar Joe Banks2020-11-15-0/+14
| | | | | | |
| | | * | | | Add Kubernetes deployment manifestGravatar Joe Banks2020-11-15-0/+21
| | |/ / / / | |/| | | |
| | | * | | 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.
* | | | Refactor Silence Class ImportsGravatar Hassan Abouelela2020-11-22-10/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactors imports of silence class to be more inline with the original import structure. Signed-off-by: Hassan Abouelela <[email protected]>
* | | | Improves Channel Converter UsabilityGravatar Hassan Abouelela2020-11-22-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Allows spaces in channel name - Allows channel name to have any capitalization - Fixed inherited class to general Converter class Signed-off-by: Hassan Abouelela <[email protected]>
* | | | Adds VoiceChannel MuteGravatar Hassan Abouelela2020-11-16-47/+132
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds an optional channel parameter to silence and unsilence commands, and adds ability to silence voice channels. TODO: New Tests Signed-off-by: Hassan Abouelela <[email protected]>
* | | | Merge branch 'master' into voicechannel-muteGravatar Hassan Abouelela2020-11-16-281/+351
|\| | |
| * | | Merge pull request #1284 from ↵Gravatar Joe Banks2020-11-15-281/+351
| |\ \ \ | | | | | | | | | | | | | | | python-discord/sebastiaan/features/move-ci-to-github-actions
| | * | | Use GHCR for the site container in docker-composeGravatar Sebastiaan Zeeff2020-11-15-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The docker-compose file should pull the site container from the GitHub Container Registry instead of DockerHub, as the latter will not receive new container images. Snekbox currently still pulls from DockerHub as it's not yet migrated to GHCR. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Remove DockerHub from GitHub ActionsGravatar Sebastiaan Zeeff2020-11-15-11/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't use DockerHub anymore; let's remove it! Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Push container to both DockerHub and GHCRGravatar Sebastiaan Zeeff2020-11-14-2/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To make the transition easier, we push the Docker container to both DockerHub and the GitHub Container Registry. I've also added a secondary tag by short commit SHA. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Add documentation to GitHub Actions stepsGravatar Sebastiaan Zeeff2020-11-14-5/+25
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Remove codeql analysis as it had little effectGravatar Sebastiaan Zeeff2020-11-14-32/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The codeql analysis action we had proved to add little value to our test suite and has been removed. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Set flake8 action checkName to correct value Gravatar Sebastiaan Zeeff2020-11-14-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | The `checkName` value of this action needs to have the same value as the name of the job. Co-authored-by: Joe Banks <[email protected]>
| | * | | Stop Checkout Actions from persisting credentialsGravatar Sebastiaan Zeeff2020-11-14-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, the Checkout Actions persists the credentials in the environment. As our Actions will also run for PRs made from a fork, we don't want to persist credentials in such a way. I've also: - Ported a comment on PIP_USER and pre-commit from the azure configs - Removed unnecessary id for the pre-commit caching step Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Add CI dependency coveralls to our PipfileGravatar Sebastiaan Zeeff2020-11-14-4/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The dependency `coveralls` was installed directly in GitHub Actions, as it's not required for local dev environments. However, it's a small package and there's value in keeping all our dependency specifications in one place. That's why I've moved it to the [dev] section of our Pipfile. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | Migrate CI Pipeline to GitHub ActionsGravatar Sebastiaan Zeeff2020-11-14-251/+265
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've migrated our Azure CI Pipeline to GitHub Actions. While the general workflow is the same, there are a few changes: - `flake8` is no longer run by `pre-commit`, but rather by a separate action that adds annotations to the GH Action results page. - As we no longer have need for xml-formatted coverage files, the xmlrunner for unittest has been removed as a dependency. Instead, we now publish our coverage results to coveralls.io. - We use version 2 of docker's GitHub Action build-and-push flow, which is split over multiple steps instead of one. - I have changed the badges to GitHub Actions and coveralls.io badges. Note: Because we accept PRs from forks, we need to be a bit careful with our secrets. While we do use the `pull_request_target` event, we should not expose secrets in steps that run code from the repository. Signed-off-by: Sebastiaan Zeeff <[email protected]>
* / / / Implements Channel ConverterGravatar Hassan Abouelela2020-11-15-0/+40
|/ / / | | | | | | | | | | | | | | | | | | Adds a converter that can decipher more forms of channel mentions, to lay foundation for voice channel muting. Signed-off-by: Hassan Abouelela <[email protected]>
* | | CI: disable 'continueOnError'Gravatar kwzrd2020-11-13-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After #1219, we started to encounter issues with the cache being corrupted and CI failing due to 'pre-commit' not being installed after restore. Although it doesn't seem likely that this could have been the culprit, the issues began appearing shortly after merging the PR. Let's see what happens if we disable it.
* | | CI: invalidate environment cacheGravatar kwzrd2020-11-13-1/+1
| | | | | | | | | | | | The cache was corrupted again for unknown reasons.
* | | Merge pull request #1272 from python-discord/Hemlock/voice-gate-pingGravatar Daniel Brown2020-11-12-3/+100
|\ \ \ | | | | | | | | Voice Gate: notify new unverified users in the verification channel
| * | | Config: ensure 2 blank lines between classesGravatar kwzrd2020-11-11-0/+3
| | | | | | | | | | | | | | | | | | | | Previous changes reduced the spacing to 1 blank line, which is inconsistent with the prevailing style.
| * | | Merge CI dep cache bump from 'master' branchGravatar kwzrd2020-11-11-1/+1
| |\ \ \ | |/ / / |/| | |
* | | | CI: invalidate dependency cacheGravatar kwzrd2020-11-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | The cache became corrupted for reasons what we were not able to figure out, causing the pre-commit step to fail when the environment was retrieved from the cache. By changing the key, we force cache rebuild.
| * | | Merge CI changes from 'master' branchGravatar kwzrd2020-11-10-5/+9
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1273 from ↵Gravatar Dennis Pham2020-11-08-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/sebastiaan/bugs/codeblock-language-detection Detect codeblock language specifiers with special characters
| * \ \ \ Merge branch 'master' into sebastiaan/bugs/codeblock-language-detectionGravatar Dennis Pham2020-11-08-1/+5
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #1148 from Jack92829/masterGravatar Senjan212020-11-07-0/+3
|\ \ \ \ \ | | | | | | | | | | | | New guilds tag
| * | | | | Update guilds.mdGravatar Jack928292020-09-29-3/+2
| | | | | | | | | | | | | | | | | | Not to sure of the title to give it but I think the content is a bit more in line with the servers other tags
| * | | | | Update guilds.mdGravatar Jack928292020-09-26-4/+3
| | | | | |
| * | | | | Merge branch 'master' into masterGravatar Matteo Bertucci2020-09-23-2462/+2436
| |\ \ \ \ \
| * | | | | | Update guilds.mdGravatar Jack928292020-09-06-1/+1
| | | | | | |
| * | | | | | Update and rename ServersTag.md to guilds.mdGravatar Jack928292020-09-06-5/+5
| | | | | | |
| * | | | | | Add files via uploadGravatar Jack928292020-09-06-0/+5
| | | | | | |
* | | | | | | Merge pull request #1219 from python-discord/bug/ci/cache-continue-on-errorGravatar Senjan212020-11-07-1/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | CI: avoid failing whole job if a cache task fails
| * | | | | | | CI: avoid failing whole job if a cache task failsGravatar MarkKoz2020-10-06-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Restoring from cache is non-critical. The CI can recover if cache tasks fail.
| | | | * | | | Detect codeblock language with special charactersGravatar Sebastiaan Zeeff2020-11-06-1/+1
| |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The regex we use to detect codeblocks did not recognize language specifiers that use a dash, a plus, or a dot in their name. As there are valid language specifiers, such as python-repl and c++, that use those characters, I've changed the regex to reflect that. The character set used now reflects the characters used in language specifiers in highlight.js. Signed-off-by: Sebastiaan Zeeff <[email protected]>
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-25 15:08:05 +0000