DEV: Don't install eslint and babel-eslint (#624)
authorJarek Radosz <jradosz@gmail.com>
Fri, 29 Apr 2022 10:18:10 +0000 (12:18 +0200)
committerGitHub <noreply@github.com>
Fri, 29 Apr 2022 10:18:10 +0000 (12:18 +0200)
Core has eslint in its dependencies, so there's no need for global eslint as far I can tell.

image/discourse_dev/Dockerfile

index 8967a6654611afa8aac0530158786b3036c006cd..96373b2f50b0d203e43835632a6b5014453c7b0d 100644 (file)
@@ -38,8 +38,7 @@ ADD ensure-database /etc/runit/1.d/ensure-database
 RUN wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | sudo apt-key add - &&\
     echo "deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google.list &&\
     apt update &&\
-    apt install -y google-chrome-stable firefox-esr &&\
-    npm install -g eslint babel-eslint
+    apt install -y google-chrome-stable firefox-esr
 
 # Install & Configure MailHog (https://github.com/mailhog/MailHog)
 RUN wget -qO /tmp/mailhog https://github.com/mailhog/MailHog/releases/download/v1.0.1/MailHog_linux_amd64\