-
Notifications
You must be signed in to change notification settings - Fork 146
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
feat(deps): update to @typescript-eslint/*
v6
#945
Conversation
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.
looks good to me!
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.
LGTM, thanks for taking care of this!
All tests are failing, can you take a look @MichaelDeBoey ? |
49ee565
to
00682e5
Compare
00682e5
to
a27659c
Compare
🎉 This PR is included in version 7.0.0-beta.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@@ -51,7 +51,8 @@ | |||
"type-check": "tsc --noEmit" | |||
}, | |||
"dependencies": { | |||
"@typescript-eslint/utils": "^5.62.0" | |||
"@typescript-eslint/scope-manager": "^6.21.0", | |||
"@typescript-eslint/utils": "^6.21.0" |
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.
Shouldn't this be:
"@typescript-eslint/utils": "^8.12.0"
?
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.
No, because this pull request is upgrading them to v6, not v8
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.
Ah! I see there is a #947 v7 and presumably v8 will happen at some point.
First part of migration to latest
typescript-eslint
versionCC/ @G-Rath @JoshuaKGoldberg
Closes #946