Skip to content

Commit

Permalink
Removing unnecessary npm commands from DockerFile
Browse files Browse the repository at this point in the history
We no longer need to install node-sass since it's been replaced by dart-sass. Also, no need to 'npm install' here since that happens in docker-compose.yml. When 'npm install' runs here, I get integrity check errors. Not sure why, but might have something to do with node version mismatch.
  • Loading branch information
alexsielicki committed Jan 24, 2022
1 parent c175595 commit 1f9e94d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions docker/compose/slycat-compose/slycat-client/DockerFile
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ RUN git config --global http.sslVerify false
RUN git clone --depth 1 https://github.com/sandialabs/slycat.git
WORKDIR /usr/src/slycat/slycat
#RUN npm config set registry http://registry.npmjs.org/
RUN npm config set strict-ssl false
RUN NODE_TLS_REJECT_UNAUTHORIZED=0 npm install node-sass --save-dev
RUN NODE_TLS_REJECT_UNAUTHORIZED=0 npm install
# RUN npm config set strict-ssl false
# RUN NODE_TLS_REJECT_UNAUTHORIZED=0 npm install node-sass --save-dev
# RUN NODE_TLS_REJECT_UNAUTHORIZED=0 npm install
# RUN npm rebuild node-sass
# No need to rebuild node-sass anymore since it was replaced with dart-sass
CMD ls

0 comments on commit 1f9e94d

Please sign in to comment.