Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Run release action against triggering tag instead of checking out branch #1864

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

jhiemstrawisc
Copy link
Member

Previously, the action would trigger on a tag like v7.12.3 and would then use regex to checkout v7.12.x, which it would use for generating goreleaser artifacts.

If I'm parsing things correctly, this means that creating a tag and then adding extra commits on top of 7.12.x could cause us to build the wrong thing if the tag is pushed after the extra commits.

Instead, this just checks out whatever github ref triggered the action and builds that.

Previously, the action would trigger on a tag like `v7.12.3` and would then use
regex to checkout `v7.12.x`, which it would use for generating goreleaser artifacts.

If I'm parsing things correctly, this means that creating a tag and then adding extra
commits on top of 7.12.x _could_ cause us to build the wrong thing.

Instead, this just checks out whatever github ref triggered the action and builds that.
@jhiemstrawisc jhiemstrawisc added bug Something isn't working infrastructure GitHub Actions, Release management, and CI labels Jan 6, 2025
@jhiemstrawisc jhiemstrawisc added this to the v7.13.0 milestone Jan 6, 2025
@jhiemstrawisc jhiemstrawisc requested a review from Saartank January 6, 2025 17:33
@jhiemstrawisc jhiemstrawisc linked an issue Jan 6, 2025 that may be closed by this pull request
@Saartank Saartank merged commit 5131f57 into PelicanPlatform:main Jan 7, 2025
23 of 25 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working infrastructure GitHub Actions, Release management, and CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release action should checkout a tag, not a branch
2 participants