Skip to content
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

ci: configure prerelease-suffix #2990

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

rootulp
Copy link
Collaborator

@rootulp rootulp commented Dec 4, 2023

Configure GoReleaser's prerelease-suffix: https://goreleaser.com/customization/git/#git

If the prerelease-suffix is not configured, then GoReleaser will treat v1.5.0-rc0 as more recent than v1.5.0. This is problematic because if the real release is cut after the release candidate, then GoReleaser won't attach pre-built binaries to the real release. This just happened on celestia-app v1.5.0 so ideally this saves ya'll some time when celestia-node starts publishing pre-built binaries.

Motivating issue: celestiaorg/celestia-app#2897

Configure GoReleaser's prerelease-suffix:
https://goreleaser.com/customization/git/#git

If the prerelease-suffix is not configured, then GoReleaser will treat
`v1.5.0-rc0` as more recent than `v1.5.0`. This is problematic because
if the real release is cut after the release candidate, then GoReleaser
won't attach pre-built binaries to the real release. This just happened
on celestia-app v1.5.0 so ideally this saves ya'll some time when
celestia-node starts publishing pre-built binaries.

Motivating issue:
celestiaorg/celestia-app#2897
@rootulp rootulp self-assigned this Dec 4, 2023
@github-actions github-actions bot added the external Issues created by non node team members label Dec 4, 2023
@rootulp rootulp added the kind:ci CI related PRs label Dec 4, 2023
@rootulp rootulp marked this pull request as ready for review December 4, 2023 22:38
@ramin ramin enabled auto-merge (squash) December 5, 2023 09:39
@ramin ramin merged commit b7b6350 into celestiaorg:main Dec 5, 2023
19 of 21 checks passed
@rootulp rootulp deleted the rp/prerelease-suffix branch December 5, 2023 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external Issues created by non node team members kind:ci CI related PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants