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

sc-5117 Add goreleaser #89

Merged
merged 2 commits into from
Jun 14, 2022
Merged

sc-5117 Add goreleaser #89

merged 2 commits into from
Jun 14, 2022

Conversation

pdeziel
Copy link
Collaborator

@pdeziel pdeziel commented Jun 10, 2022

This adds the goreleaser GitHub action to publish releases of the rVASP program.

@pdeziel pdeziel requested a review from bbengfort June 10, 2022 14:22
env:
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
GPG_FINGERPRINT: ${{ steps.import_gpg.outputs.fingerprint }}
HOMEBREW_TAP_GITHUB_TOKEN: ${{secrets.HOMEBREW_TAP_GITHUB_TOKEN}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think we need to get homebrew involved for the rvasp CLI at this stage so you can probably remove this secret.

Copy link
Collaborator

@bbengfort bbengfort left a comment

Choose a reason for hiding this comment

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

You're right that we'll have to tag a version to see if this works or not; but it's a good start!

A couple of minor comments inline.

args: release --rm-dist
env:
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
GPG_FINGERPRINT: ${{ steps.import_gpg.outputs.fingerprint }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you configure the GitHub secrets in settings with everything needed here or does that still have to be done?

- -v

# Custom ldflags templates.
ldflags: -s -w -X main.version={{.Version}} -X main.commit={{.Commit}} -X main.date={{.Date}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does the cmd/rvasp package have these variables? I think we have to set them in pkg.version?

# Footer for the release body.
footer: |
### About
The rVASP tool is a binary command line application that is used to run and
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍🏽

@pdeziel pdeziel merged commit 7daa896 into main Jun 14, 2022
@pdeziel pdeziel deleted the sc-5117 branch June 14, 2022 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants