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

Feat/#175 homebrew release #180

Merged
merged 11 commits into from
Oct 13, 2023

Conversation

sFritsch09
Copy link
Contributor

I needed to edit the build pipeline because using the build steps for gorelease you need to have the Pro version: https://goreleaser.com/customization/builds/#import-pre-built-binaries

Anyway I kept the same binary structure as before and the pipeline is much shorter with goreleaser now.

✅ Added Homebrew with GoReleaser
✅ Edit build pipeline and tested workflow

In order to let GoRelease push the artefacts you need to add GITHUB_TOKEN secret:
The minimum permissions the GITHUB_TOKEN should have to run this are write:packages

Copy link
Collaborator

@andrew-farries andrew-farries left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @sFritsch09 👍

I've made some suggestions.

  • I'm going to create the homebrew-pgroll repository. [repo]

.goreleaser.yaml Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
.goreleaser.yaml Show resolved Hide resolved
.goreleaser.yaml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
.goreleaser.yaml Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
sFritsch09 and others added 3 commits October 9, 2023 13:39
@sFritsch09
Copy link
Contributor Author

Pleasure @andrew-farries, it's my first PR so far, appreciate your input!
I guess everything is in order following the implementation of your suggestions. Please inform me if there are any remaining matters to address :)

docs/README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@philkra
Copy link
Contributor

philkra commented Oct 10, 2023

Hi @sFritsch09 , thank you very much for your contribution! We would love to send you some swag, could you reach out to me via email: philip@xata.io to coordinate? Cheers, Philip

Copy link
Collaborator

@andrew-farries andrew-farries left a comment

Choose a reason for hiding this comment

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

I've removed the remaining formatting changes from docs/README.md and updated the repo's settings.yaml file.

This looks good to go now 🚢

Thanks for your contribution @sFritsch09! ❤️

@andrew-farries andrew-farries merged commit e87c639 into xataio:main Oct 13, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants