tools: make update-eslint.sh work with npm@9

Make the `update-eslint.sh` script work with `npm@9`. PR-URL: #46088 Refs: https://github.com/nodejs/node/actions/runs/3814364920/jobs/6488613583#step:3:64 Reviewed-By: Ruy Adorno <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
nodejs · Jan 31, 2023 · 609df01 · 609df01
1 parent d8ce990
commit 609df01
Showing 1 changed file with 34 additions and 6 deletions.
diff --git a/tools/update-eslint.sh b/tools/update-eslint.sh
@@ -21,13 +21,41 @@ rm -rf node_modules/eslint
 
     "$NODE" "$NPM" init --yes
 
-    "$NODE" "$NPM" install --global-style --no-bin-links --ignore-scripts eslint
-    # Uninstall plugins that we want to install so that they are removed from devDependencies.
-    # Otherwise --production will cause them to be skipped.
-    (cd node_modules/eslint && "$NODE" "$NPM" uninstall --ignore-scripts eslint-plugin-jsdoc eslint-plugin-markdown @babel/core @babel/eslint-parser @babel/plugin-syntax-import-assertions)
-    (cd node_modules/eslint && "$NODE" "$NPM" install --no-save --no-bin-links --ignore-scripts --production --omit=peer eslint-plugin-jsdoc eslint-plugin-markdown @babel/core @babel/eslint-parser @babel/plugin-syntax-import-assertions)
+    "$NODE" "$NPM" install \
+    --ignore-scripts \
+    --install-strategy=shallow \
+    --no-bin-links \
+    eslint
+    # Uninstall plugins that we want to install so that they are removed from
+    # devDependencies. Otherwise --omit=dev will cause them to be skipped.
+    (
+        cd node_modules/eslint
+        "$NODE" "$NPM" uninstall \
+        --install-links=false \
+        --ignore-scripts \
+        eslint-plugin-jsdoc \
+        eslint-plugin-markdown \
+        @babel/core \
+        @babel/eslint-parser \
+        @babel/plugin-syntax-import-assertions
+    )
+    (
+        cd node_modules/eslint
+        "$NODE" "$NPM" install \
+        --ignore-scripts \
+        --install-links=false \
+        --no-bin-links \
+        --no-save \
+        --omit=dev \
+        --omit=peer \
+        eslint-plugin-jsdoc \
+        eslint-plugin-markdown \
+        @babel/core \
+        @babel/eslint-parser \
+        @babel/plugin-syntax-import-assertions
+    )
     # Use dmn to remove some unneeded files.
-    "$NODE" "$NPM" exec -- [email protected] -f clean
+    "$NODE" "$NPM" exec --package=[email protected] --yes -- dmn -f clean
     # TODO: Get this into dmn.
     find node_modules -name .package-lock.json -exec rm {} \;
     find node_modules -name 'README*' -exec rm {} \;