| Commit message (Collapse) | Author | Age | Lines |
... | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* bundle build & deploy into a single workflow
* delete separate build & deploy wokrflows
* trigger status embed on the new bundled "Build & Deploy" workflow
* Call linting & build-deployment workflows from main
* keep one empty line in workflow files
* use v4 of k8s deploy
* yeet kubectl version
* use v3 of k8s-set-context
* setup kubectl config using the setup-kubectl v3 action
* use v2 of the checkout action
* add dependabot config
* add new line for dependabot.yml
* update path of the sentry_release workflow
* update path of the lint-test workflow
* specify push branch
* update the sentry_release workflow path, again
* update sentry_release's trigger to workflow_call
* update lint-test's trigger to workflow_call
* remove extra line at the end
* unify usage of quotes
* update concurrency groups of reusable local workflows
* appease our dear linter with a blank line
* always run status embed
* Revert "always run status embed"
This reverts commit 2f69fd77b212337ff45bf7815783030bc6099452.
* remove colons after workflow_call
* remove empty line at the end
* Revert "remove empty line at the end"
This reverts commit 223bef4159c76f1968878fe312b5fbc16544549b.
* echo event name in status embed
* try running status embed on bot ci/cd wf
* run ci/cd on push to main
* reinstate the Lint & Test trigger for status embed
* rename main wf name to "CI"
This also triggers status embed on that name
* add group-workflows to the list of branches
* remove test step
* trigger status embed on ci only
* trigger main on main branch only
* remove concurrency groups in lint-test & sentry-release workflows
|
| |
| |
| | |
Co-authored-by: ChrisJL <[email protected]>
|
|\| |
|
| |
| |
| |
| | |
This reverts commit 2f8efdc646492707f6a9647fcbc4d71678181b90.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* make status embed reusable
* invoke status embed from lint-test
* trigger status embed from within deploy workflow
* trigger status embed from build wf
* update extension of referenced reusable workflow
* add github.sha to construct the concurrency group
* always call the status_embed from build & deploy
|
| | |
|
|/ |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
This is caused by an upstream issue with setuptools 60.* (via virtualenv) changeing the default to using the setuptools-embedded distutils rather than the stdlib distutils, which breaks within pip's isolated builds.
This is explained quite well here https://github.com/pre-commit/pre-commit/issues/2178#issuecomment-1002163763
|
| |
|
| |
|
| |
|
|\ |
|
| | |
|
| | |
|
| |
| |
| |
| | |
Since our project is licensed under the MIT License, we can't be using any dependencies in our project. This commit adds a step to the CI lint phase that will use pip-licenses to verify that all the installed packages are part of an ALLOWED_LICENSE variable. This variable is currently set to be every license we currently use. We opted to use an allowlist instead of a denylist to make sure that new licenses are reviewed by hand and added to that variable.
|
|/
|
|
|
|
|
|
|
|
| |
This concurrency rule means that workflow actions will be cancelled if a new set of actions are triggered on the same ref. A ref is either a PR or a branch itself.
Doing this means that if someone pushes to a PR multiple times in quick succession, workflows won't get queued up. Instead, only the workflows from the most recent changes will run, as this is all we really care about anyway.
I see the benefits of this as twofold.
1. The author gets faster feedback about the most recent change pushed, rather than have to wait for all previous runs to completed
2. Other contributors don't need to wait for a queue of redundant workflows to finish.
|
|
|
|
| |
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
| |
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
| |
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
|
|
|
|
| |
Switches the test runner from unittest to pytest, to allow the usage of
plugins such as xdist. This commit also adds pytest-cov purely as a
generator for .coverage files.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
|
|
|
| |
Updates Pipfile and pipenv lock file to poetry equivalents in
CODEOWNERS.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
|
|
|
| |
Updates the Dockerfile, pre-commit, CI, and documentation to reflect the
new dependency manager. Dockerfile is also updated to 3.9.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
| |
GITHUB_TOKEN has now enough permissions to be used for pushing to GHCR, so we don't need PAT anymore.
|
| |
|
| |
|
| |
|
|
|
|
| |
python-discord/.github
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
GitHub Actions has an implicit status condition, `success()`, that
is added whenever an `if` condition lacks a status function check
of its own. In this case, while the upload step did check for the
outcome of the previous "always" step, it did not have an actual
status check and, thus, only ran on success.
Since we always want to upload the artifact, even if other steps
failed, I've added the "always" status function now.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit introduces the same custom status embed as is already being
used for Sir Lancebot. The default embeds GitHub sends are disabled, as
they were causing slight issues with rate limits from time to time.
It works like this:
- The Lint & Test workflow stores an artifact with PR information, if we
are linting/testing a PR.
- Whenever we reach the end of a workflow run sequence, a status embed
is send with the conclusion status.
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\ |
|
| | |
|
|/ |
|
| |
|
| |
|