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: fix goreleaser check deprecation warning (backport #3795) #3797

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Aug 20, 2024

Closes #3793 by resolving this deprecation notice


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

@mergify mergify bot requested a review from a team as a code owner August 20, 2024 17:12
@mergify mergify bot requested review from staheri14 and rach-id August 20, 2024 17:12
@github-actions github-actions bot added the bot item was created by a bot label Aug 20, 2024
@rootulp rootulp enabled auto-merge (squash) August 20, 2024 17:32
@rootulp
Copy link
Collaborator

rootulp commented Aug 20, 2024

proto-check-breaking failing is not caused by this PR. Pretty sure it was caused by: 02b604d#diff-04c475aa44bf3f902c435d878f1dc5ea9240aed2d74a4d881b519e8817b3cbf5R37 cc: @ninabarbakadze

@rootulp rootulp merged commit f378ff5 into v2.x Aug 20, 2024
32 of 35 checks passed
@rootulp rootulp deleted the mergify/bp/v2.x/pr-3795 branch August 20, 2024 23:48
rootulp added a commit that referenced this pull request Aug 22, 2024
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
mergify bot pushed a commit that referenced this pull request Aug 22, 2024
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)
rootulp added a commit that referenced this pull request Aug 22, 2024
#3813)

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
https://github.com/goreleaser/goreleaser-cross/pull/77<hr>This is an
automatic backport of pull request #3806 done by
[Mergify](https://mergify.com).

Co-authored-by: Rootul P <rootulp@gmail.com>
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.

2 participants