-
Notifications
You must be signed in to change notification settings - Fork 26
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
Fix goreleaser configuration #30
Fix goreleaser configuration #30
Conversation
file_name_template: >- | ||
{{- .ProjectName }}_ | ||
{{- .Version }}_ | ||
{{- if eq .Os "darwin" }}macOS | ||
{{- else if eq .Os "linux" }}Linux | ||
{{- else if eq .Os "openbsd" }}OpenBSD | ||
{{- else if eq .Os "netbsd" }}NetBSD | ||
{{- else if eq .Os "freebsd" }}FreeBSD | ||
{{- else if eq .Os "dragonfly" }}DragonFlyBSD | ||
{{- else }}{{ .Os }}{{ end }}- | ||
{{- if eq .Arch "amd64" }}64bit | ||
{{- else if eq .Arch "386" }}32bit | ||
{{- else if eq .Arch "arm" }}ARM | ||
{{- else if eq .Arch "arm64" }}ARM64 | ||
{{- else }}{{ .Arch }}{{ end }} |
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.
This keeps compatibility with the previous format, see: https://github.com/ngehrsitz/cob/releases/tag/v0.0.8
Not sure if the complexity is worth it though.
This depends on #28 being merged first because the golangci-lint version in this branch is too old for |
Looks good, but the linting test is failing. |
Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 1 to 5. - [Release notes](https://github.com/goreleaser/goreleaser-action/releases) - [Commits](goreleaser/goreleaser-action@v1...v5) --- updated-dependencies: - dependency-name: goreleaser/goreleaser-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
705a422
to
9a656bd
Compare
I rebased and it should work now. |
uses: goreleaser/goreleaser-action@v5 | ||
with: | ||
distribution: goreleaser | ||
version: v1.21.2 |
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 opted to pin this version since goreleaser appears to be introducing breaking changes somewhat frequently https://goreleaser.com/deprecations/
Thanks |
Description of changes:
This fixes the broken goreleaser configuration and will allow publishing new versions. Uptaking Go 1.21 as build version was also necessary because of goreleaser/goreleaser#2066.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.