-
-
Notifications
You must be signed in to change notification settings - Fork 405
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
build: Manage Dockerfile with Dofigen #1014
Conversation
@taorepoara is attempting to deploy a commit to the OpenStatus Team on Vercel. A member of the Team first needs to authorize it. |
# Temp solution for bun | ||
RUN bun install --ignore-scripts | ||
# install | ||
FROM oven/bun@sha256:eb409bed239c3adff079a6b71283f151e802d66b99f643ba7a71e1be7d3da513 AS install |
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.
@taorepoara do we really need the hash?
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.
The hash gives you the control on the FROM
images. Dofigen does it by default, but I'll add a flag to avoid that: lenra-io/dofigen#278
workdir: /app/ | ||
# Copy project | ||
bind: | ||
- package.json |
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.
I m ashamed by that part our monorepo is 😱
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@thibaultleouay Let me know if there is things to change |
Type of change
Description
Generate the server Dockerfile with Dofigen in order to optimize the resulting image size and the build time.
A picture tells a thousand words (if any)
Before this PR
{Please add a screenshot here}
After this PR
{Please add a screenshot here}
Related Issue (optional)
#1012