refactor(ts): migrate babel-eslint-plugin-* to TS - #17950
Conversation
Sorry, something went wrong.
| } | ||
|
|
||
| if (parent.type === "ExportNamedDeclaration") { | ||
| return node.id.name; |
There was a problem hiding this comment.
This file is renamed to is-default-export such that we don't have to deal with the typing issue here from destructuring at all.
Sorry, something went wrong.
| }, | ||
| "homepage": "https://babel.dev/", | ||
| "peerDependencies": { | ||
| "eslint": "^9.0.0 || ^10.0.0" |
There was a problem hiding this comment.
We add the eslint peer deps constraint to align with other eslint plugins.
Sorry, something went wrong.
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/61383 |
Sorry, something went wrong.
| }, | ||
| "homepage": "https://github.com/babel/babel/tree/master/eslint/babel-eslint-plugin-development-internal", | ||
| "peerDependencies": { | ||
| "@babel/eslint-parser": "workspace:^", |
There was a problem hiding this comment.
This eslint plugin does not work on any early stage features and therefore we remove the @babel/eslint-parser from the peer deps. Of course it should still work with @babel/eslint-parser.
Sorry, something went wrong.
|
commit: d4de73b |
Sorry, something went wrong.
| "new-cap": "off", | ||
| "no-undef": "off", | ||
| "object-curly-spacing": "off", | ||
| "no-unused-expressions": "off", |
There was a problem hiding this comment.
This is a bug fix. The object-curly-spacing rule has already been removed as ESLint has deprecated that rule.
Sorry, something went wrong.
In this PR we migrate babel-eslint-plugin-* and fixed all typing errors. The only introduced @ts-expect-error is for an upstream typing issue.
We also fixed small bugs detected by tsc and added new tests.