Move chown into same RUN as npm install

github · Apr 10, 2023 · aa70600 · aa70600
1 parent 7726451
commit aa70600
Showing 1 changed file with 6 additions and 10 deletions.
diff --git 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 #