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: use goreleaser-cross for goreleaser-check command (backport #3806) #3813

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Aug 22, 2024

Closes #3805

goreleaser v2.2.0 deprecated the name_template field in favor of version_template. Our goreleaser-check command used a GH action that was using goreleaser v2.2.0 so we fixed the deprecation in #3797 but the make prebuilt-binary command uses goreleaser-cross which was using a previous version of goreleaser that fails for the new field.

This PR updates the goreleaser-check command to use the goreleaser-cross Docker image so that the check command and prebuilt-binary command uses the same version.

Note: we'll need to update name_template to version_template when we upgrade to Go 1.23.0 because that will use goreleaser-cross 1.23.0 which bumped to goreleaser 2.22.0. See goreleaser/goreleaser-cross#77


This is an automatic backport of pull request #3806 done by Mergify.

Closes #3805

[goreleaser
v2.2.0](https://github.com/goreleaser/goreleaser/releases/tag/v2.2.0)
deprecated the `name_template` field in favor of `version_template`. Our
goreleaser-check command used a GH action that was using goreleaser
v2.2.0 so we fixed the deprecation in
#3797 but the make
prebuilt-binary command uses goreleaser-cross which was using a previous
version of goreleaser that fails for the new field.

This PR updates the goreleaser-check command to use the goreleaser-cross
Docker image so that the check command and prebuilt-binary command uses
the same version.

Note: we'll need to update `name_template` to `version_template` when we
upgrade to Go 1.23.0 because that will use goreleaser-cross 1.23.0 which
bumped to goreleaser 2.22.0. See
goreleaser/goreleaser-cross#77

(cherry picked from commit 13f3098)
@mergify mergify bot requested a review from a team as a code owner August 22, 2024 17:03
@mergify mergify bot requested review from cmwaters and staheri14 August 22, 2024 17:03
@github-actions github-actions bot added the bot item was created by a bot label Aug 22, 2024
@rootulp rootulp enabled auto-merge (squash) August 22, 2024 17:03
@rootulp rootulp merged commit 48173df into v2.x Aug 22, 2024
33 checks passed
@rootulp rootulp deleted the mergify/bp/v2.x/pr-3806 branch August 22, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot item was created by a bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants