Skip to content

feat(deps): update to @typescript-eslint/* v8 #955

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

MichaelDeBoey
Copy link
Member

Last part of migration to latest typescript-eslint version

CC/ @G-Rath @JoshuaKGoldberg


Closes #948

@MichaelDeBoey MichaelDeBoey added the v7 Next major v7 label Nov 1, 2024
@MichaelDeBoey MichaelDeBoey self-assigned this Nov 1, 2024
@@ -214,6 +212,7 @@ export function detectTestingLibraryUtils<

const originalNodeName =
isImportSpecifier(importedUtilSpecifier) &&
ASTUtils.isIdentifier(importedUtilSpecifier.imported) &&
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra check is needed because of changes from typescript-eslint/typescript-eslint#9972: imported can now be Identifier | StringLiteral, which was only Identifier before

@MichaelDeBoey MichaelDeBoey linked an issue Nov 1, 2024 that may be closed by this pull request
@MichaelDeBoey MichaelDeBoey force-pushed the update-to-typescript_eslint-v8 branch from 83c107e to fd32bc5 Compare November 1, 2024 19:29
@MichaelDeBoey MichaelDeBoey marked this pull request as ready for review November 1, 2024 19:29
@MichaelDeBoey MichaelDeBoey requested a review from Belco90 November 1, 2024 19:29
@Belco90
Copy link
Member

Belco90 commented Nov 9, 2024

I should be able to review this next week. Thanks.

@MichaelDeBoey MichaelDeBoey force-pushed the update-to-typescript_eslint-v8 branch from fd32bc5 to b7d0890 Compare November 19, 2024 00:05
@MichaelDeBoey
Copy link
Member Author

@Belco90 anything you still want me to do before this can be merged or is it good to go?

Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, amazing work. Thanks @MichaelDeBoey!

@Belco90 Belco90 merged commit e015071 into beta Nov 19, 2024
26 checks passed
@Belco90 Belco90 deleted the update-to-typescript_eslint-v8 branch November 19, 2024 11:37
Copy link

🎉 This PR is included in version 7.0.0-beta.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

Upgrade @typescript-eslint to v8
2 participants