Skip to content

Commit

Permalink
tools: remove unneeded tool in update-eslint.sh
Browse files Browse the repository at this point in the history
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: #40995
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
Trott authored and danielleadams committed Dec 13, 2021
1 parent 5400b79 commit 6525226
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions tools/update-eslint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,6 @@ rm -rf node_modules/eslint
"$NODE" "$NPM" exec -- [email protected] -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 -- [email protected] .
)

mv eslint-tmp/node_modules/eslint node_modules/eslint
Expand Down

0 comments on commit 6525226

Please sign in to comment.