You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 13, 2019. It is now read-only.
Right now, the deps folder is in the .dockerignore and therefore not being copied over to the build image which means that all deps are retrieved from scratch.
I don't see any reason to prevent them from being copied over as it will speed up the build time significantly if this step (fetching deps) can be bypassed because the deps folder in the docker image has been warmed already.
For example, in my CI server, I run mix deps.get && mix test as a step before running mix docker.shipit. By removing the deps folder from .dockerignore, i cut the build time in half.
The text was updated successfully, but these errors were encountered:
tlvenn
added a commit
to tlvenn/mix_docker
that referenced
this issue
Apr 18, 2017
Right now, the
deps
folder is in the.dockerignore
and therefore not being copied over to the build image which means that all deps are retrieved from scratch.I don't see any reason to prevent them from being copied over as it will speed up the build time significantly if this step (fetching deps) can be bypassed because the deps folder in the docker image has been warmed already.
For example, in my CI server, I run
mix deps.get && mix test
as a step before runningmix docker.shipit
. By removing thedeps
folder from.dockerignore
, i cut the build time in half.The text was updated successfully, but these errors were encountered: