Skip to content
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

chore: bump typescript-eslint to v8 #1623

Conversation

JoshuaKGoldberg
Copy link
Contributor

@JoshuaKGoldberg JoshuaKGoldberg commented Jul 6, 2024

👋 Hi! Coming over from typescript-eslint/typescript-eslint#9501: we're working on typescript-eslint v8 and would like to try the beta out on plugins such as eslint-plugin-jest.

I'm sending this draft PR as a reference to see what issues we might give you in upgrading and to try to be helpful. If this is annoying noise to you, please forgive me, I'll withdraw the PR. But I hope this is useful on your end - and please let me know if you have any feedback! ❤️

'@typescript-eslint/ban-types': 'error',
'@typescript-eslint/no-empty-object-type': 'error',
'@typescript-eslint/no-unsafe-function-type': 'error',
'@typescript-eslint/no-wrapper-object-types': 'error',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ban-types is removed in typescript-eslint v8. Its default options are now split among those three rules.

requiresTypeChecking?: boolean;
}

export const createRule = ESLintUtils.RuleCreator<JestPluginDocs>(name => {
Copy link
Contributor Author

@JoshuaKGoldberg JoshuaKGoldberg Jul 6, 2024

Choose a reason for hiding this comment

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

https://typescript-eslint.io/blog/announcing-typescript-eslint-v8-beta/#custom-rule-metadocs-types

In theory there's no hard need in plugins to mention requiresTypeChecking anymore... except, in practice eslint-doc-generator reads it. Also this repo uses it for more doc generation configuration:

ruleListSplit: 'meta.docs.requiresTypeChecking',

@SimenB
Copy link
Member

SimenB commented Jul 6, 2024

If this is annoying noise to you, please forgive me, I'll withdraw the PR.

Not at all! Please keep the engagement up 🙂

@notaphplover
Copy link

@JoshuaKGoldberg are you working on this? typescript-eslint 8 is released and it would be great to update this repo to add support for it. If you are too busy to go for it, would it be ok to submit another PR to add support for typescript-eslint 8?

@@ -106,7 +106,7 @@
"typescript": "^5.0.4"
},
"peerDependencies": {
"@typescript-eslint/eslint-plugin": "^6.0.0 || ^7.0.0",
"@typescript-eslint/eslint-plugin": "*",

Choose a reason for hiding this comment

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

Are we sure every @typescript-eslint/eslint-plugin would be a valid peer?

Copy link

github-actions bot commented Aug 3, 2024

🎉 This issue has been resolved in version 28.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@JoshuaKGoldberg JoshuaKGoldberg deleted the typescript-eslint-v8 branch August 4, 2024 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants