chore: Use ESLint v9 in development env #2775
Conversation
🦋 Changeset detectedLatest commit: 6e393d5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@2nofa11 Once this PR is merged, please rebase your PR 🙏 |
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
There was a problem hiding this comment.
Wow, thank you! Apart from two questions (which don't necessarily need to be resolved in context of this PR), this looks good to me.
I think this PR fixes #2756, right? So maybe @2nofa11 you want to have a look, too?
I’d love to take a look!
I’m still getting up to speed with the codebase, so I might have some beginner-level questions 😭
| "@typescript-eslint/parser": "^7.0.0 || ^8.0.0" | ||
| }, | ||
| "peerDependenciesMeta": { | ||
| "@typescript-eslint/parser": { | ||
| "optional": true | ||
| } |
There was a problem hiding this comment.
Can you please add another changeset (with a minor version update) explaining this change?
There was a problem hiding this comment.
I have added the changeset 👍
This PR will change the development environment to use ESLint v9.
After looking into the documentation build errors in #2770, I found that a lot of changes are needed.
I want to address those issues first with this PR.
Also, some tests were failing for the script-indent rule when using typescript-eslint v8, so I fixed those as well.