-
Notifications
You must be signed in to change notification settings - Fork 2
feat: build Ubuntu arm64 #12
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
350879a to
1e9920d
Compare
1e9920d to
61c8791
Compare
.github/workflows/publish.yaml
Outdated
| --tag $IMAGE_NAME:${RELEASE_VERSION} \ | ||
| --tag $IMAGE_NAME:latest \ | ||
| --tag $IMAGE_NAME:${RELEASE_VERSION}-${{ matrix.tag_suffix }} \ | ||
| --tag $IMAGE_NAME:latest-${{ matrix.tag_suffix }} \ |
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 kind of need the image name to be latest still, otherwise down stream will not be able to pull latest
Were you able to execute this locally?
| args: '' | ||
| - platform: 'windows-latest' | ||
| args: '' | ||
| docker_platform: 'windows/amd64' # adjust based on your image build support |
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.
Why is this removed?
| paradigm-flood = "^0.3.1" | ||
| lxml = {extras = ["html-clean"], version = "^5.3.1"} | ||
| aioprometheus = "^23.12.0" | ||
| connectorx = "^0.4.3" |
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.
What is this dependency?
|
@xiuxiuxar can you add in the arm check? But i think other than that this is good to go to see if ti releases. |
8ball030
left a comment
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.
Interesting lets see what happens, thanks alot for the pr!
No description provided.