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

fix: goreleaser stuff #61

Merged
merged 2 commits into from
Jan 4, 2025
Merged

fix: goreleaser stuff #61

merged 2 commits into from
Jan 4, 2025

Conversation

jmeridth
Copy link
Member

@jmeridth jmeridth commented Jan 3, 2025

Fixes #12

  • rebuild .goreleaser.yml from scratch to meet new version (goreleaser init)
    • copy over pieces we need from old config
    • verified good with goreleaser check
  • change goreleaser github action to fire on newly published releases, not new tags

Our new release github action will create the release and do all the templating based on conventional commits pr titles and auto-labelling. Now goreleaser will see the newly published release and attach the new asset builds to the new release.

Was tested in https://github.com/jmeridth/test-reusable-release-with-goreleaser

@Copilot Copilot bot review requested due to automatic review settings January 3, 2025 20:17
@jmeridth jmeridth requested a review from a team as a code owner January 3, 2025 20:17

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again, by re-requesting a review.

@jmeridth jmeridth self-assigned this Jan 3, 2025
@github-actions github-actions bot added the fix label Jan 3, 2025
eddie-knight
eddie-knight previously approved these changes Jan 3, 2025
@jmeridth jmeridth marked this pull request as draft January 4, 2025 02:04
@jmeridth jmeridth marked this pull request as ready for review January 4, 2025 05:14
@jmeridth jmeridth changed the title fix: version is goreleaser's version not our repo's fix: goreleaser stuff Jan 4, 2025
- [x] rebuild .goreleaser.yml from scratch to meet new version (goreleaser init)
  - copy over pieces we need from old config
  - verified good with `goreleaser check`
- [x] change goreleaser github action to fire on newly published releases, not new tags

Our new release github action will create the release and do all the templating based on conventional commits pr titles and auto-labelling.  Now goreleaser will see the newly published release and attach the new asset builds to the new release.

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth merged commit 879a300 into main Jan 4, 2025
8 checks passed
@jmeridth jmeridth deleted the jm_fix_goreleaser branch January 4, 2025 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix Release Pipeline
2 participants