Skip to content

Commit f89d194

Browse files
Trotttargos
authored andcommitted
tools: improve update-eslint.sh
* Pin version numbers when using `npx`. This means we can be careful about updating what we run. (Thinking about the recent eslint-scope thing.) * Add removeNPMAbsolutePaths to get rid of unused fields in package.json files. These unused fields can cause unnecessary churn, as at least one contains an absolute path that will be different for each developer. PR-URL: #21819 Reviewed-By: Yuta Hiroto <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
1 parent af1530e commit f89d194

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

tools/update-eslint.sh

+4-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,10 @@ npm install --no-bin-links --production --no-package-lock eslint-plugin-markdown
2020
cd ../..
2121

2222
# Use dmn to remove some unneeded files.
23-
npx dmn -f clean
23+
npx [email protected] -f clean
24+
# Use removeNPMAbsolutePaths to remove unused data in package.json.
25+
# This avoids churn as absolute paths can change from one dev to another.
26+
2427

2528
cd ..
2629
mv eslint-tmp/node_modules/eslint node_modules/eslint

0 commit comments

Comments
 (0)