From 106219e5b53fcef6f5ea7a362a90971cd6f6b594 Mon Sep 17 00:00:00 2001
From: Guillaume Grossetie <ggrossetie@gmail.com>
Date: Sat, 7 Nov 2020 12:31:43 +0100
Subject: [PATCH] Remove the extra "deps" step

This step can be done as part of the build (faster and more reliable).
---
 Dockerfile | 19 +++----------------
 1 file changed, 3 insertions(+), 16 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 298b440700..ad5ebb5c59 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,25 +1,12 @@
-FROM node:14-alpine AS deps
-
-ARG REACT_APP_INCLUDE_GTAG=false
-
-RUN mkdir /opt/node_app && chown node:node /opt/node_app
-WORKDIR /opt/node_app
-
-USER node
-
-COPY package.json package-lock.json ./
-RUN npm install --no-optional && npm cache clean --force
-ENV PATH /opt/node_app/node_modules/.bin:$PATH
+FROM node:14-alpine AS build
 
 WORKDIR /opt/node_app
 COPY . .
+RUN npm i --no-optional
 
-FROM node:14-alpine AS build
-
+ARG REACT_APP_INCLUDE_GTAG=false
 ARG NODE_ENV=production
 
-WORKDIR /opt/node_app
-COPY --from=deps /opt/node_app .
 RUN npm run build:app:docker
 
 FROM nginx:1.17-alpine