Dev: Migrate to ESLint 9, flat config #32471
Conversation
| "lint-addons": "eslint examples/jsm --ext .js --ignore-pattern libs --ignore-pattern ifc", | ||
| "lint-examples": "eslint examples --ext .html", | ||
| "lint-docs": "eslint docs --ignore-pattern prettify.js", | ||
| "lint-editor": "eslint editor --ignore-pattern libs", | ||
| "lint-playground": "eslint playground --ignore-pattern libs", | ||
| "lint-manual": "eslint manual --ignore-pattern 3rdparty --ignore-pattern prettify.js --ignore-pattern shapefile.js", | ||
| "lint-test": "eslint test --ignore-pattern vendor", | ||
| "lint-utils": "eslint utils --ignore-pattern prettify --ignore-pattern fuse", | ||
| "lint-addons": "eslint examples/jsm", | ||
| "lint-examples": "eslint examples", | ||
| "lint-docs": "eslint docs", | ||
| "lint-editor": "eslint editor", | ||
| "lint-playground": "eslint playground", | ||
| "lint-manual": "eslint manual", | ||
| "lint-test": "eslint test", | ||
| "lint-utils": "eslint utils", |
There was a problem hiding this comment.
Since ignore patterns are included in the config, now, it's possible we can merge these commands together and let the user specify any sub folder checks themselves. I can look into adding these back if they're preferred, though.
My "lint" scripts are set to eslint . so the whole project is validated.
| 'no-async-promise-executor': 'off', | ||
| 'no-empty': 'off', | ||
|
|
||
| // 'jsdoc/check-types': 'error', |
There was a problem hiding this comment.
When this is enabled there are some errors in the JSDoc comments that are caught. I'll leave it to another contribution to re-enable and fix these. It may be worth enabling the JSDoc "recommended" settings above to see if there are any rules worth keeping from there, too.
| 'no-useless-escape': 'off', | ||
| 'no-case-declarations': 'off', | ||
| 'no-cond-assign': 'off', | ||
| 'getter-return': 'off', | ||
| 'no-async-promise-executor': 'off', | ||
| 'no-empty': 'off', |
There was a problem hiding this comment.
I turned these off because they seemed to be new rules and were causing lint errors / warnings. It may be worth considering adjusting the codebase and turning these on.
| BigInt: 'readonly', | ||
| BigUint64Array: 'readonly' |
There was a problem hiding this comment.
These lines have been newly added
📦 Bundle sizeFull ESM build, minified and gzipped.
🌳 Bundle size after tree-shakingMinimal build including a renderer, camera, empty scene, and dependencies.
|
1 participant
Related issue: -
Description
The ESLint 9 configs have moved to a new flat config format. I've migrated some of my other projects so I thought I'd try it here, trying to keep the rules the same as much as I can. Settings like "jsdoc" rules have since changed to a separate plugin and in the future some of the styling rules will change to a separate code style plugin. There is an open issue at the mdcs plugin here to migrate, as well.