Skip to content
This repository was archived by the owner on Nov 14, 2024. It is now read-only.

chore: add release job and goreleaser config #18

Merged
merged 1 commit into from
Sep 15, 2021
Merged

Conversation

johnstcn
Copy link
Member

No description provided.

@johnstcn johnstcn requested a review from jawnsy September 14, 2021 15:55
@johnstcn johnstcn self-assigned this Sep 14, 2021
permissions:
actions: none
checks: none
contents: write
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess this needs contents: write to upload the release? Seems unfortunate that we have to give write permissions to the source code for this token

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, that's my assumption here.

Copy link
Contributor

@jawnsy jawnsy left a comment

Choose a reason for hiding this comment

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

Nice, thanks for taking care of this! It was on my to-do list, but you got to it first 🏃

uses: goreleaser/goreleaser-action@v2
with:
# either 'goreleaser' (default) or 'goreleaser-pro'
distribution: goreleaser
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there some way of making preview releases in PRs so that we know everything will build correctly before we tag it?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we could add something like this as a step in the build stage:

goreleaser release --snapshot --skip-publish --rm-dist

@johnstcn johnstcn merged commit ac58589 into main Sep 15, 2021
@johnstcn johnstcn deleted the cj/ch14279/goreleaser branch September 15, 2021 15:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants