-
Notifications
You must be signed in to change notification settings - Fork 52
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
Add goreleaser #146
Add goreleaser #146
Conversation
.github/workflows/releaser.yml
Outdated
username: ${{ github.repository_owner }} | ||
password: ${{ secrets.GITHUB_TOKEN }} | ||
|
||
- name: Publich ref name |
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.
typo
.github/workflows/releaser.yml
Outdated
- name: Login to GHCR | ||
uses: docker/login-action@v2 | ||
with: | ||
registry: ghcr.io |
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.
let's use docker hub, i'll help set this up
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.
wdyt?
probably docker hub is more common and easier to use, as you don't need to be logged into github (it's more easily accessible)
I've created https://hub.docker.com/r/flashbots/suave-geth and added env vars to the repo settings
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.
Did you set DOCKERHUB_USERNAME
and DOCKERHUB_TOKEN
?
nice work! |
Makefile
Outdated
-v `pwd`/sysroot:/sysroot \ | ||
-w /go/src/$(PACKAGE_NAME) \ | ||
ghcr.io/goreleaser/goreleaser-cross:v1.19.5 \ | ||
release --clean --release-notes /build/changelog.md |
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 think it'd be better to create a draft release here, and manually edit the description and publish when ready
.github/workflows/releaser.yml
Outdated
username: ${{ github.repository_owner }} | ||
password: ${{ secrets.GITHUB_TOKEN }} | ||
|
||
- name: Publich ref name |
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.
- name: Publich ref name | |
- name: Public ref name |
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.
Updated to Log tag name
password: ${{ secrets.GITHUB_TOKEN }} | ||
|
||
- name: Publich ref name | ||
run: echo "Build for tag ${{ github.ref_name }}" |
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.
does this work? in the past I've had to use this way:
https://github.com/flashbots/mev-boost/blob/develop/.github/workflows/release.yaml#L20-L26
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.
Yes, check this execution log https://github.com/ferranbt/suave-geth/actions/runs/7446372136/job/20256482872
📝 Summary
This PR introduces
goreleaser
to build releases forlinux
,darwin
andwindows
withcgo
enabled. It also producesdocker
images and pushes them toghcr
.📚 References