-
Notifications
You must be signed in to change notification settings - Fork 39
Add support for ESLint v9 #79
base: master
Are you sure you want to change the base?
Conversation
It looks like typescript-eslint is supporting flat configs now: |
👋 @gund are you still planning on working on this soon? |
As in: I'm happy to hop in and write a PR, but don't want to step on your toes. 🙂 |
@JoshuaKGoldberg - thought I was commenting on this earlier 😂 - hoping you can hop in and get this over the line if @gund is tied up! Thanks to you both for the hard work! |
@gund now that ESLint v9 is out and typescript-eslint/typescript-eslint#8211 is closed in upstream, would you consider finishing and publishing this PR? (or publishing a pre-release version?) As an alternative, there is also apparently an open proposal in |
Hi @gund, any updates about a future release for ESlint V9 ? 👀 |
Hey @gund we are looking to move to eslint v9 but this is the only remaining plugin really holding us back. |
For anyone wanting a rule like the one included in
Diff: -import deprecation from 'eslint-plugin-deprecation';
// ...
plugins: {
'eslint-comments': eslintComments,
- deprecation: fixupPluginRules(
- /** @type {import("eslint").ESLint.Plugin} */ (
- /** @type {unknown} */ (deprecation)
- ),
- ),
},
// ...
rules: {
- // Warn about usage of deprecated APIs
- // https://github.com/gund/eslint-plugin-deprecation
- 'deprecation/deprecation': 'warn',
+ // Warn about usage of deprecated APIs
+ // https://typescript-eslint.io/rules/no-deprecated/
+ '@typescript-eslint/no-deprecated': 'warn', Caveat: In my first testing, it felt like it may be slower than |
This resolves this for us, ty. |
Add support for ESLint v9 while maintaining backwards compatibility with all previously supported ESLint versions.
Closes #78
This PR is cannot be finished until: