Skip to content

Commit

Permalink
Try a different approach to node_modules
Browse files Browse the repository at this point in the history
  • Loading branch information
dstufft committed Apr 16, 2016
1 parent 753d18b commit 116962a
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Dockerfile
Expand Up @@ -38,7 +38,7 @@ RUN set -x \
&& apt-get update \
&& apt-get install gcc make libpq-dev libffi-dev --no-install-recommends -y \
&& rm -rf node_modules \
&& npm install -g bower node-gyp gulp-cli \
&& npm install -g gulp-cli \
&& npm install \
&& pip install -U pip setuptools \
&& pip install -r requirements/dev.txt \
Expand Down
12 changes: 10 additions & 2 deletions docker-compose.yml
Expand Up @@ -20,8 +20,16 @@ camo:
app:
build: .
volumes:
- .:/app/
- /app/node_modules
# We specify all of these directories instead of just . because we want to
# avoid having ./node_modules from the host OS being shared with the docker
# container, and since there's no way to exclude a directory, the only way
# to make this work is to share multiple, smaller directories. These cover
# the important things that we want to share, but changes to requirements
# or any file not in these directories will require a rebuild.
- ./dev:/app/dev
- ./docs:/app/docs
- ./warehouse:/app/warehouse
- ./tests:/app/tests
environment:
WAREHOUSE_ENV: development
WAREHOUSE_TOKEN: insecuretoken
Expand Down

0 comments on commit 116962a

Please sign in to comment.