Skip to content

Pinned the goreleaser version to 2#1991

Merged
jhiemstrawisc merged 3 commits intoPelicanPlatform:mainfrom
turetske:clearup-deprecated-goreleaser-warnings
Feb 10, 2025
Merged

Pinned the goreleaser version to 2#1991
jhiemstrawisc merged 3 commits intoPelicanPlatform:mainfrom
turetske:clearup-deprecated-goreleaser-warnings

Conversation

@turetske
Copy link
Collaborator

@turetske turetske commented Feb 5, 2025

Very simple PR, fixes in the containers as well since they build from source.

@turetske turetske added infrastructure GitHub Actions, Release management, and CI internal Internal code improvements, not user-facing labels Feb 5, 2025
@turetske turetske added this to the v7.14 milestone Feb 5, 2025
@turetske turetske linked an issue Feb 5, 2025 that may be closed by this pull request
@turetske turetske removed the request for review from jhiemstrawisc February 5, 2025 20:58
@turetske turetske force-pushed the clearup-deprecated-goreleaser-warnings branch from 2a66619 to 8238134 Compare February 5, 2025 21:31
Copy link
Member

@jhiemstrawisc jhiemstrawisc left a comment

Choose a reason for hiding this comment

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

This gets rid of the goreleaser warning about using "version 0", but I don't see that it fixed the first item listed in the linked issue. When I try to build Pelican with these changes I still see:

    • DEPRECATED:  snapshot.name_template  should not be used anymore, check https://goreleaser.com/deprecations#snapshotname_template for more info

@turetske turetske assigned turetske and jhiemstrawisc and unassigned turetske Feb 7, 2025
Copy link
Member

@jhiemstrawisc jhiemstrawisc left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@jhiemstrawisc jhiemstrawisc merged commit 8dbab66 into PelicanPlatform:main Feb 10, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure GitHub Actions, Release management, and CI internal Internal code improvements, not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tidy up deprecation warnings in goreleaser

2 participants