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

Add separate steps in GitHub action for snapshots and release builds #59

Merged
merged 1 commit into from
Dec 14, 2022

Conversation

simu
Copy link
Member

@simu simu commented Dec 13, 2022

Summary

The goreleaser v4 GitHub action has completely removed support for automatically doing snapshot builds for commits which aren't associated with a tag. Since goreleaser's --snapshot doesn't have any smarts to do a non-snapshot build for commits which are tagged, we split the goreleaser step into separate steps for actions running against a tag and actions running against master.

Follow-up to #58

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.
  • Link this PR to related issues.

The goreleaser v4 GitHub action has completely removed support for
automatically doing snapshot builds for commits which aren't associated
with a tag. Since goreleaser's `--snapshot` doesn't have any smarts to
do a non-snapshot build for commits which are tagged, we split the
goreleaser step into separate steps for actions running against a tag
and actions running against master.
@simu simu added the ignore label Dec 13, 2022
@simu simu requested a review from a team December 13, 2022 16:31
@simu simu merged commit e58da00 into master Dec 14, 2022
@simu simu deleted the fix/build-snapshot-packages branch December 14, 2022 10:45
simu added a commit that referenced this pull request Dec 14, 2022
Despite my claim in #59, goreleaser does have logic to automatically
enable snapshots for dirty commits. This commit reverts most of the
changes from #59 and instead enables `--auto-snapshot`.

Additionally, `goreleaser build` doesn't actually build the Linux
packages, so it wasn't very useful in the first place.
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.

2 participants