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 Feb 1, 2022
1 parent 017cdcd commit e5cccdf
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 e5cccdf

Please sign in to comment.