From e5cccdfd74d162db843046d0df4c79350f3a2c6b Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sun, 28 Nov 2021 08:33:16 -0800 Subject: [PATCH] tools: remove unneeded tool in update-eslint.sh MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The tool to remove absolute paths from package.json files is no longer necessary. It appears that npm no longer stores these paths, or at least not in a way that causes the kind of churn we saw in the past. PR-URL: https://github.com/nodejs/node/pull/40995 Reviewed-By: Luigi Pinca Reviewed-By: Ruben Bridgewater Reviewed-By: Tobias Nießen Reviewed-By: James M Snell --- tools/update-eslint.sh | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/update-eslint.sh b/tools/update-eslint.sh index ba2b63cd4e9df7..a36189dd2d8ffe 100755 --- a/tools/update-eslint.sh +++ b/tools/update-eslint.sh @@ -27,9 +27,6 @@ rm -rf node_modules/eslint "$NODE" "$NPM" exec -- dmn@2.2.2 -f clean # TODO: Get this into dmn. find node_modules -name .package-lock.json -exec rm {} \; - # Use removeNPMAbsolutePaths to remove unused data in package.json. - # This avoids churn as absolute paths can change from one dev to another. - "$NODE" "$NPM" exec -- removeNPMAbsolutePaths@1.0.4 . ) mv eslint-tmp/node_modules/eslint node_modules/eslint