feat: Add goreleaser #41
Conversation
|
As this is bringing in an outside package, it is worth commenting on that package. GoReleaser uses MIT license, which while not called out in our contributor's guide, is close enough in alignment to the ApacheV2 we use to be acceptable. From a governance and licensing standpoint, LGTM. |
bradbeam
force-pushed
the
packages
branch
3 times, most recently
from
96e8fbf
to
072fd37
Compare
This introduces goreleaser as the means by which we will release vhs. This should include packaging vhs as a debian as well as producing a container image. Signed-off-by: Brad Beam <brad.beam@stormforge.io>
|
I think this is mostly there, wouldn't mind getting it merged to play around with cutting some prereleases and see how the process goes. |
There was a problem hiding this comment.
Since this PR merges the Dockerfiles in the repo / deprecates the original vhs-dev container, does it make sense to remove the make dev entry from the Makefile? As a broader question, does it still make sense to have a standardized development container, or no? Other than that, looks good to me! 👍
Co-authored-by: Stephan <zeisss@moinz.de>
This introduces goreleaser as the means by which we will release vhs.
This should include packaging vhs as a debian as well as producing
a container image.
There's still quite a bit of cleanup and whatnot that needs to happen, but this should set the direction.
Signed-off-by: Brad Beam brad.beam@stormforge.io