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 go releaser #38

Merged
merged 1 commit into from
Oct 1, 2019
Merged

Add go releaser #38

merged 1 commit into from
Oct 1, 2019

Conversation

zph
Copy link
Collaborator

@zph zph commented Sep 21, 2019

(Builds on #37, will rebase after prior PRs land)

This PR sets up GoReleaser as a mechanism for building binaries and setting up a homebrew formula for use as brew install tj/mmake.

@tj tj added Priority: Low Low priority issue and removed Priority: Low Low priority issue labels Sep 23, 2019
@zph
Copy link
Collaborator Author

zph commented Oct 1, 2019

@tj 👋 . This is a port of the goreleaser config I setup on my fork and after renaming things... I think it's right for this repo! Please take a look and see if it looks good to you.

@tj
Copy link
Owner

tj commented Oct 1, 2019

LGTM! I think some of those are defaults, so we could probably remove them but no harm either way

@zph
Copy link
Collaborator Author

zph commented Oct 1, 2019

Agreed on some of them being defaults 👍since they don't bother you, I'll leave them in for the explicitness :D. Later me will thank me for the choice.

@zph zph merged commit 98ab244 into master Oct 1, 2019
@zph zph deleted the add-go-releaser branch October 1, 2019 19:25
@tj
Copy link
Owner

tj commented Oct 2, 2019

True haha, good point :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants