aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Add inline cache manifest to docker imageGravatar Sebastiaan Zeeff2020-11-20-0/+1
| | | | | We need to attach an inline cache manifest to the image we build and push to the GHCR to be able to use the registry as a cache source for subsequent builds. I've added it to the build workflow configuration.configuration.
* Update workflow status badges in readmeGravatar Sebastiaan Zeeff2020-11-19-1/+7
| | | Our two workflows, Lint and Build, now have separate status badges that link to the latest results from that workflow.
* Merge pull request #520 from python-discord/sebastiaan/backend/fix-flake8-checkGravatar Leon Sandøy2020-11-19-78/+85
|\ | | | | Fix flake8 check in CI and separate Lint and Build jobs into separate workflows
| * Split lint and build jobs into separate workflowsGravatar Sebastiaan Zeeff2020-11-16-57/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | I've separated the lint and build jobs into two separate workflows: - Lint: .github/workflows/lint.yaml - Build: .github/workflows/build.yaml The main difference is that the Build workflow will be triggered if the Lint workflow completes while it's been run on the "master" branch. The build job will check if the Lint run was successful and if it were, it actually builds the container, pushes it the GHCR, and triggers the deployment to our kubernetes cluster.
| * Fix lint errors introduced by persistence removalGravatar Sebastiaan Zeeff2020-11-16-8/+8
| | | | | | | | | | | | | | | | | | | | | | We ripped some features out of the bot to make the migration to kubernetes easier. However, that process left us with some linting errors that will play up if anyone makes a PR. I've added "noqa" to the valid TODO lines and commented out lines that introduced now unused names or tried to use now undefined names. This should have no influence on the bot, as it does not touch code that is actually being run at the moment.
| * Ensure flake8 runs correctly in Pull Request checkGravatar Sebastiaan Zeeff2020-11-16-13/+11
|/ | | | | | | | | | | | | | | | | | | | | | | | | | Unfortunately, the flake8 action we were using from the marketplace required us to use the `pull_request_target` event, which runs in the context of the target repository to protect secrets. However, this also meant that flake8 would run on files already merged into our master branch, not the actual changes made in teh PR! That's obviously pretty useless as a guard against merging linting errors into our repository. This change sidesteps the issue by removing the marketplace action and replacing it by a direct `flake8` run command. To make sure error output ends up as a GitHub Actions Annotation, we ask `flake8` to format its error messages using the correct GitHub Workflow Command format: ::error file={filename},line={line},col={col}::{message} Whenever something is printed to stdout/stderr in that format, GitHub Actions will automatically interpret it as the "register error message" workflow command. Since this doesn't require a GitHub Token with "write" permissions, we can now switch back to the safer `pull_request` event that gets a read-only GitHub Token.
* Merge pull request #517 from ↵Gravatar Leon Sandøy2020-11-15-11/+3
|\ | | | | | | | | python-discord/sebastiaan/backend/remove-dockerhub-push Remove DockerHub from GitHub Actions
| * Remove DockerHub from GitHub ActionsGravatar Sebastiaan Zeeff2020-11-15-11/+3
|/ | | | | We're no longer using DockerHub in our deployment for SeasonalBot, so it does not make sense to keep pushing containers there.
* Change CI badge to GitHub ActionsGravatar Sebastiaan Zeeff2020-11-15-1/+1
|
* Merge pull request #516 from python-discord/kubernetes-deployGravatar Joe Banks2020-11-14-3/+38
|\
| * Merge build and deploy stagesGravatar Joe Banks2020-11-14-19/+2
| |
| * Add Kubernetes deployment manifestGravatar Joe Banks2020-11-14-0/+21
| |
| * Add Kubernetes deployment stage to GitHub ActionsGravatar Joe Banks2020-11-14-1/+32
|/
* Merge pull request #515 from python-discord/feature/non-persistenceGravatar Leon Sandøy2020-11-14-90/+19
|\ | | | | Remove persistence requirements from Seasonalbot
| * Remove make_persistent import from bot extensionsGravatar Jeremiah Boby2020-11-14-4/+8
| |
| * Merge branch 'master' into feature/non-persistenceGravatar Jeremiah Boby2020-11-14-68/+144
| |\ | |/ |/|
* | Fix typo in GitHub workflowGravatar Joe Banks2020-11-14-1/+1
| |
* | Merge pull request #514 from python-discord/sebastiaan/backend/move-ci-to-ghaGravatar Leon Sandøy2020-11-14-68/+144
|\ \ | | | | | | Migrate to GitHub Actions and GHCR
| * | Add documentation to GitHub Actions workflowGravatar Sebastiaan Zeeff2020-11-14-6/+24
| | | | | | | | | | | | | | | | | | I've added a bunch of comments to document some of the steps. I've also removed the unnecessary `persist-credentials: false` option from the build phase.
| * | Migrate to GitHub Actions and GHCRGravatar Sebastiaan Zeeff2020-11-14-68/+126
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are currently migrating our backend to a Kubernetes cluster. As this requires changes to our CI anyway, this is a good time to also migrate our CI Pipeline to GitHub Actions and start using the GitHub Container Registry to push our builds. Changes compared to the old situation: - flake8 linting errors will now show up as GH Actions annotations - containers are tagged with both "latest" and a short commit SHA Note: This version of the workflow still pushes to DockerHub IN ADDITION TO GHCR. This is to make the transition between the two deployment systems as seamless as possible.
| * Remove references to deprecated methods in cog initGravatar Jeremiah Boby2020-11-14-5/+6
| |
| * Raise NotImplementedError on write/read to configGravatar Jeremiah Boby2020-11-14-4/+51
| |
| * Remove persist bot utilGravatar Jeremiah Boby2020-11-14-69/+0
| |
| * Remove unused arrow importGravatar Jeremiah Boby2020-11-14-1/+0
| |
| * Remove CandyCollection cogs from loaded cogsGravatar Jeremiah Boby2020-11-14-1/+0
| |
| * Remove HacktoberStats cog from loaded cogsGravatar Jeremiah Boby2020-11-14-1/+0
| |
| * Revert autoformatter changesGravatar Jeremiah Boby2020-11-14-2/+8
| |
| * Don't load Monster survey cogGravatar Jeremiah Boby2020-11-14-1/+0
| |
| * Remove persistent methods from evergreen branding managerGravatar Jeremiah Boby2020-11-14-58/+2
|/
* Merge pull request #496 from quanta-kt/candyGravatar Joe Banks2020-10-31-130/+106
|\ | | | | Fix candy mini-game to group leader-board scores correctly
| * Merge branch 'master' into candyGravatar Joe Banks2020-10-31-65/+75
| |\ | |/ |/|
* | Merge pull request #478 from python-discord/discordpy-15Gravatar Joe Banks2020-10-31-65/+75
|\ \ | | | | | | Bump 'discord.py' to 1.5.1 and use explicit intents
| * \ Merge branch 'master' into discordpy-15Gravatar Joe Banks2020-10-31-145/+536
| |\ \ | |/ / |/| |
| * | Intents: use 'default' factoryGravatar kwzrd2020-10-31-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | Since we disable both privileged intents, we can just construct an instance with them already off. As Senjan notes, this will also future-proof us against any privileged intents added in the future. Co-authored-by: Senjan21 <[email protected]>
| * | Intents: document which intents are privilegedGravatar kwzrd2020-10-31-2/+2
| | |
| * | Intents: disable privileged 'Members' intentGravatar kwzrd2020-10-31-0/+1
| | | | | | | | | | | | | | | | | | | | | With 'd.py' 1.5.1, the Member converter will now lazily fetch the member object. It is believed that this removes our need for the intent.
| * | Deps: bump 'discord.py' to 1.5.1 & re-lockGravatar kwzrd2020-10-31-51/+52
| | | | | | | | | | | | | | | Bugfix release: https://discordpy.readthedocs.io/en/latest/whats_new.html#v1-5-1
| * | Intents: call classmethod on class, not intermediate instanceGravatar kwzrd2020-10-05-1/+1
| | |
| * | Intents: initial setupGravatar kwzrd2020-10-05-0/+10
| | | | | | | | | | | | | | | | | | | | | For now, we require the privileged 'Guild Members' intent, to maintain all current functionality (e.g. Member convertors working with IDs). In the future, we may look into disabling this intent.
| * | Deps: bump 'discord.py' to 1.5 & re-lockGravatar kwzrd2020-10-04-22/+22
| | |
| | * Merge branch 'master' into candyGravatar Quanta2020-10-26-136/+421
| | |\ | |_|/ |/| |
* | | Merge pull request #479 from Anubhav1603/bug_ecGravatar Senjan212020-10-24-44/+50
|\ \ \ | | | | | | | | [Bug] emoji command
| * | | removed _ from command nameGravatar Anubhav16032020-10-14-1/+1
| | | |
| * | | improved readbility of code by breaking one liner into if elseGravatar Anubhav16032020-10-13-8/+8
| | | |
| * | | Update bot/exts/evergreen/emoji_count.pyGravatar Anubhav2020-10-11-1/+4
| | | | | | | | | | | | Co-authored-by: Rohan Reddy Alleti <[email protected]>
| * | | Update bot/exts/evergreen/emoji_count.pyGravatar Anubhav2020-10-11-1/+4
| | | | | | | | | | | | Co-authored-by: Rohan Reddy Alleti <[email protected]>
| * | | removed len,\n and now passing ctx.guild.emojisGravatar Anubhav16032020-10-09-9/+9
| | | |
| * | | added s in emoji_countGravatar Anubhav16032020-10-07-2/+2
| | | |
| * | | added typing.tupleGravatar Anubhav16032020-10-07-3/+4
| | | |
| * | | Merge branch 'master' into bug_ecGravatar Thomas Petersson2020-10-07-7/+113
| |\ \ \
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-22 19:57:15 +0000