-
-
Notifications
You must be signed in to change notification settings - Fork 805
Building with docker #125
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
Building with docker #125
Conversation
Do you mean this? Currently this has only impact on the golang version. What exactly do you mean with building agains a tag/commit. That the gomod-cap check will be executed with the go.mod files from these commits? |
Codecov Report
@@ Coverage Diff @@
## master #125 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 39 39
Lines 1400 1400
=====================================
Hits 1400 1400Continue to review full report at Codecov.
|
Yes.
Okay, I will take a close look at that and leave my review tomorrow :) |
|
Thanks! Would you mind if I add you as a outside collaborator? Then I could add you as reviewer. With the plugin-template something like this should already be possible: $ make GOTIFY_VERSION=v1.2.1 checkIs that what you meant?` |
That sounds great:) Thanks With the plugin-template something like this should already be possible: Let's track this in gotify/plugin-template#2 Yeah it does have a check-go-mod that checks the dependency against a server version, but it cannot update go.mod automatically to fit another version of server, so practically using only prepared commands in Makefile you cannot update go.mod to fit another server version in the build script other than what was committed in the VCS. In my netlify repo . make GOTIFY_VERSION=v1.3.0 update-go-mod
make build # build for 1.3.0
make GOTIFY_VERSION=v1.4.0 update-go-mod
make build # build for 1.4.0 |
eternal-flame-AD
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds great to me:)
Since go 1.12 is very likely to be coming out this week I am considering whether to use it for the 1.3.0
|
In gotify/build@43beb35 I've added automatic building of gotiy/build docker images. They now are tagged like this |
The arm7 docker image wasn't really used and the base image gotify/server used got deprecated.
@eternal-flame-AD Could you have a look at this PR and gotify/website#6?