-
Notifications
You must be signed in to change notification settings - Fork 672
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
Build simd images as part of the release process #1533
Build simd images as part of the release process #1533
Conversation
.github/workflows/build-simd.yml
Outdated
@@ -0,0 +1,39 @@ | |||
name: Create and publish simd Docker image | |||
|
|||
on: workflow_dispatch |
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.
should we make this push to master/main also can make it on pull_request to test in the PR
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 can add on pull_request and then remove before merging, ideally we'll actually just want this is the release task.
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.
Thanks @chatton, much appreciated!
@damiannolan since the build/push works as expected (the image appears here) I've moved the action into the release.yml file. Now any time we release an image of that tag will be built and pushed. |
Description
Task which needs to be run manually to build and push the simd docker image.
closes: #XXXX
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes