-
Notifications
You must be signed in to change notification settings - Fork 41
Docker: fix openmina dockerfile and slightly improve it #1190
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
Conversation
|
|
||
| COPY . . | ||
| # Build with cache mount | ||
| RUN --mount=type=cache,target=/usr/local/cargo/registry \ |
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.
docker build -t openmina/openmina:6945ec6b .
[+] Building 399.3s (21/21) FINISHED
I guess it is fine for now.
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.
2358b91 to
2866dc4
Compare
80dcc04 to
98920c2
Compare
8209382 to
a0221d1
Compare
98920c2 to
b1a8e9e
Compare
b1a8e9e to
191ed99
Compare
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.
LGTM
Always build the docker images in CI, but do not publish it except on main, develop or a tag, as before.