-
Notifications
You must be signed in to change notification settings - Fork 6k
feat(ci): semi-automate code-server-aur update #5462
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ Coverage Diff @@
## main #5462 +/- ##
=======================================
Coverage 72.44% 72.44%
=======================================
Files 30 30
Lines 1673 1673
Branches 366 366
=======================================
Hits 1212 1212
Misses 398 398
Partials 63 63 Continue to review full report at Codecov.
|
This adds a new job called aur which checkouts code-server-aur, updates the files with the new version then opens a PR into the repo.
runs-on: ubuntu-latest | ||
timeout-minutes: 10 | ||
env: | ||
GH_TOKEN: ${{ secrets.HOMEBREW_GITHUB_API_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like we should rename the secret to CDRCI_GITHUB_API_TOKEN
if we're going to use it in both places. I'll have to find the value or ask @deansheather again though if I do.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. For now this seems chill.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might want to rename the workflow since it only mentions NPM and Brew. Maybe we can simply call it publish.yaml
and even move the Docker push here.
runs-on: ubuntu-latest | ||
timeout-minutes: 10 | ||
env: | ||
GH_TOKEN: ${{ secrets.HOMEBREW_GITHUB_API_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. For now this seems chill.
Good idea! I'll do that in a followup PR. |
Description
This automates part of the release process for updating code-server AUR package. After the npm package is published, it will run this job which will update some files and open a PR to code-server-aur.
Changes
aur
job to npm/brew workflow which runs on releasesTesting
I was able to test this by adding a temporary job to
ci.yaml
and it successfully opened a PR.Next Steps
Add workflow to push to aur on pushes to master code-server-aur#13
Fixes #5366