Skip to content
This repository was archived by the owner on Jul 19, 2025. It is now read-only.

Commit 9653d52

Browse files
authored
Reorder npm install/package.json copy (#125)
This treats this more like other dependencies, and stops us from doing `npm install` on every single build.
1 parent 028d742 commit 9653d52

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ COPY Gemfile.lock /usr/src/app/
88
COPY vendor/php-parser/composer.json /usr/src/app/vendor/php-parser/
99
COPY vendor/php-parser/composer.lock /usr/src/app/vendor/php-parser/
1010

11+
COPY package.json /usr/src/app/
12+
1113
RUN curl --silent --location https://deb.nodesource.com/setup_5.x | bash - && \
1214
apt-get update && apt-get install -y nodejs python openssh-client php5-cli php5-json
1315
RUN gem install bundler --no-ri --no-rdoc && \
@@ -16,12 +18,12 @@ RUN gem install bundler --no-ri --no-rdoc && \
1618

1719
RUN mv composer.phar /usr/local/bin/composer
1820
RUN cd /usr/src/app/vendor/php-parser/ && composer install --prefer-source --no-interaction
21+
RUN npm install
1922

2023
RUN adduser app -u 9000
2124

2225
COPY . /usr/src/app
2326
RUN chown -R app .
24-
RUN npm install
2527

2628
USER app
2729

0 commit comments

Comments
 (0)