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
Cleaned up dockerfile, moved create test #50
Conversation
|
||
FROM alpine:edge | ||
FROM alpine:3.10 |
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.
Edge is the dev version, we should pin to a stable version.
mv commit0 /usr/local/bin | ||
RUN upx --lzma /usr/local/bin/* | ||
mv commit0 /usr/local/bin && \ | ||
upx --lzma /usr/local/bin/commit0 |
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.
Separating this into two steps means most of the time we only need to re-compress the commit0 binary, not the other stuff that never changes.
@@ -0,0 +1,7 @@ | |||
Dockerfile |
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.
We should make sure to make this file as strict as possible, otherwise all this stuff gets included in COPY . .
and busts the cache for all sorts of changes to various files.
No description provided.