-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade to ESLint v9 #814
Upgrade to ESLint v9 #814
Conversation
b6f0582
to
3c5b447
Compare
Pull Request Test Coverage Report for Build 10186047410Details
💛 - Coveralls |
package.json
Outdated
@@ -41,6 +42,6 @@ | |||
"typedoc": "^0.25.13", | |||
"typedoc-plugin-missing-exports": "^2.2.0", | |||
"typescript": "^5.3.3", | |||
"typescript-eslint-language-service": "^5.0.5" | |||
"typescript-eslint": "^8.0.0-alpha.13" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't mind taking taking this (it's just a dev dependency), but is there any urgency driving us to upgrade to an alpha version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just Dependabot churn, which could alternately be addressed with an ignore
3c5b447
to
5291c48
Compare
debf67d
to
9641f45
Compare
9641f45
to
c5cc13e
Compare
This is now ready to go, as typescript-eslint v8.0.0 has been released. https://typescript-eslint.io/blog/announcing-typescript-eslint-v8 |
This upgrades to ESLint v9 and an alpha version of typescript-eslint. This PR also switches from the now-deprecated eslintrc configuration file to a "flat config", changes the command line used with eslint to match, and cleans up some unused eslint-disable directives.