From aa7060080c28ee29328034c2f2276421a118f1e1 Mon Sep 17 00:00:00 2001 From: Andrew Kerr Date: Sat, 11 Mar 2023 09:46:57 -0500 Subject: [PATCH] Move chown into same RUN as npm install --- Dockerfile | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/Dockerfile b/Dockerfile index 13b264c8..c1390ae4 100644 --- a/Dockerfile +++ b/Dockerfile @@ -92,16 +92,12 @@ RUN apk add --no-cache \ ######################################## COPY dependencies/* / -############################# -# Install Dependencies # -############################# -RUN npm install && bundle install - -############################################################ -# Fix broken permissions in ast-types-flow dependency # -# Fixes https://github.com/github/super-linter/issues/3901 # -############################################################ -RUN chown -R "$(id -u)":"$(id -g)" node_modules +################################################################### +# Install Dependencies # +# The chown fixes broken uid/gid in ast-types-flow dependency # +# (see https://github.com/github/super-linter/issues/3901) # +################################################################### +RUN npm install && chown -R "$(id -u)":"$(id -g)" node_modules && bundle install ############################## # Installs Perl dependencies #