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

Fixes the release with goreleaser fixed version #110

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

sledigabel
Copy link
Contributor

Why this change?

Goreleaser has deprecated Replacements since this commit.
The github action for goreleaser uses the latest goreleaser version instead of a fixed version.

We need to address the replacements in a future commit, for now we are forcing an older version of goreleaser.
The version of goreleaser that includes the deprecation of replacements is v1.19.0.
We are rolling back now to v1.18.2.

[Replacements](https://github.com/Skyscanner/turbolift/blob/9f28424daf82663b91716b6888c12cfb6a04ca0e/.goreleaser.yml#L16-L21) have been deprecated on goreleaser since this [commit](https://github.com/goreleaser/goreleaser/pull/3589/files#diff-5563ff2aab3eb7eae7bd8557cd2c086d8373263762684283485b2f0a00a0f52a).
The github action for goreleaser unfortunately uses the latest
goreleaser version instead of a fixed version.

We need to address the replacements in a future commit, for now we are just forcing an older version of goreleaser.
The version of goreleaser that includes the deprecation of `replacements` is [v1.19.0](https://github.com/goreleaser/goreleaser/releases/tag/v1.19.0).
We are rolling back now to [v1.18.2](https://github.com/goreleaser/goreleaser/releases/tag/v1.18.2).
Copy link
Collaborator

@Dan7-7-7 Dan7-7-7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sledigabel sledigabel marked this pull request as ready for review October 13, 2023 15:34
@Dan7-7-7 Dan7-7-7 merged commit 3e0e3cf into main Nov 9, 2023
5 checks passed
sledigabel added a commit that referenced this pull request Nov 9, 2023
Since v1.19.0, goreleaser is not supporting replacements in its
configuration, rather it points you to using a template.

This change reverses the workaround from #110 and introduces
a forward-fix for this.

We still have an issue with goreleaser action using `latest` but it's by
design on their behalf.
sledigabel added a commit that referenced this pull request Feb 12, 2024
* Fix goreleaser and deprecate replacements

Since v1.19.0, goreleaser is not supporting replacements in its
configuration, rather it points you to using a template.

This change reverses the workaround from #110 and introduces
a forward-fix for this.

We still have an issue with goreleaser action using `latest` but it's by
design on their behalf.

* taps are deprecrated, changing too

* fix(goreleaser): Adding version to template

After further reading on the documentation, we were missing the Version
on the package names.
FYI, the command to test the full build, renaming and skip publish is:

```
goreleaser --snapshot --skip-publish --rm-dist
``
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants