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

Adding goreleaser #236

Merged
merged 2 commits into from
May 15, 2019
Merged

Adding goreleaser #236

merged 2 commits into from
May 15, 2019

Conversation

kensipe
Copy link
Member

@kensipe kensipe commented May 14, 2019

What type of PR is this?
/kind infrastructure

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Including circleci
Adding docs for release process

Does this PR introduce a user-facing change?:

NONE

Including circleci
Adding docs for release process
Copy link
Contributor

@alenkacz alenkacz left a comment

Choose a reason for hiding this comment

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

LGTM 👍

RELEASE.md Outdated

## Release Process

The official binaries for Kudo is created using [goreleaser](https://goreleaser.com/) for the release process through the cicleci release job. The [.goreleaser.yml](.goreleaser.yml) defines the binaries which are supported for each release.
Copy link
Contributor

Choose a reason for hiding this comment

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

are created

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 did mean "supported"... the supported and defined platforms will be created

- restore_cache:
keys:
- go-mod-v1-{{ checksum "go.sum" }}
- run: curl -sL https://git.io/goreleaser | bash
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we have this defined in makefile so one can run even locally?

Copy link
Contributor

Choose a reason for hiding this comment

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

also in the future it would be nice to run tests before release. We cannot do that now though...

Copy link
Member

Choose a reason for hiding this comment

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

shouldn't we have this defined in makefile so one can run even locally?

I generally like this idea. In this case, we'd add it to tools.go as well so we're pinned to a version of goreleaser - not sure if that fits in with how goreleaser is versioned though.

Copy link
Member

Choose a reason for hiding this comment

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

Overall fine with keeping it how it is now and doing that in a separate PR too.

- restore_cache:
keys:
- go-mod-v1-{{ checksum "go.sum" }}
- run: curl -sL https://git.io/goreleaser | bash
Copy link
Member

Choose a reason for hiding this comment

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

Overall fine with keeping it how it is now and doing that in a separate PR too.

@gerred
Copy link
Member

gerred commented May 15, 2019

lgtm

Copy link
Member

@fabianbaier fabianbaier left a comment

Choose a reason for hiding this comment

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

lgtm

RELEASE.md Outdated Show resolved Hide resolved
RELEASE.md Outdated Show resolved Hide resolved
@kensipe kensipe merged commit b92f85e into master May 15, 2019
@alenkacz alenkacz deleted the ken/releaser branch May 30, 2019 11:42
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.

None yet

4 participants