Skip to content

Add goreleaser setup#9

Open
pcarranza wants to merge 16 commits intojustwatchcom:masterfrom
pcarranza:pc/add-goreleaser-setup
Open

Add goreleaser setup#9
pcarranza wants to merge 16 commits intojustwatchcom:masterfrom
pcarranza:pc/add-goreleaser-setup

Conversation

@pcarranza
Copy link
Copy Markdown

Changes:

  • Add a couple of other dockerfiles for armv6 and arm64
  • Add a goreleaser configuration file with some assumptions that may need some reviewing
  • Simplify makefile as it's now using goreleaser to control builds

To build now just make, to release make release

Copy link
Copy Markdown

@dominikschulz dominikschulz left a comment

Choose a reason for hiding this comment

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

LGTM

@pcarranza
Copy link
Copy Markdown
Author

Humm.... seems like this is a limitation in Travis, I wonder if it makes sense to just build in travis but not generate the docker images.

Copy link
Copy Markdown

@dominikschulz dominikschulz left a comment

Choose a reason for hiding this comment

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

LGTM

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