Commit f89d194
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 <hello@hiroppy.me>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>1 parent af1530e commit f89d194
1 file changed
Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -20,7 +20,10 @@ npm install --no-bin-links --production --no-package-lock eslint-plugin-markdown | |||
| 20 | 20 | cd ../.. | |
| 21 | 21 | ||
| 22 | 22 | # Use dmn to remove some unneeded files. | |
| 23 | - npx dmn -f clean | ||
| 23 | + npx dmn@1.0.10 -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 | + npx removeNPMAbsolutePaths@1.0.4 . | ||
| 24 | 27 | ||
| 25 | 28 | cd .. | |
| 26 | 29 | mv eslint-tmp/node_modules/eslint node_modules/eslint | |
0 commit comments