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

feat: allow @typescript-eslint v8 #1636

Merged
merged 4 commits into from
Aug 3, 2024
Merged

feat: allow @typescript-eslint v8 #1636

merged 4 commits into from
Aug 3, 2024

Conversation

G-Rath
Copy link
Collaborator

@G-Rath G-Rath commented Aug 2, 2024

Unsurprisingly we don't have to actually do anything for this support - I'm not worried about the unbound-method tests as this weirdness happened before and I plan to do a new major soon which'll get all this cleaned up; everything is working fine locally

Resolves #1633
Resolves #1623

@G-Rath
Copy link
Collaborator Author

G-Rath commented Aug 2, 2024

@SimenB before you say it, yes let's do a new major so we can stop hacking around but for now this should unblock the super keen folks.

I'm in the process of converting my company's config over to ESLint v9 after which I'll do our next major and clean all this stuff up.

@G-Rath G-Rath marked this pull request as ready for review August 2, 2024 21:37
@SimenB
Copy link
Member

SimenB commented Aug 2, 2024

😃

Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

nice! (somehow didn't approve when I looked through this yesterday)

@G-Rath G-Rath merged commit fb43171 into main Aug 3, 2024
45 checks passed
@G-Rath G-Rath deleted the ts-v8 branch August 3, 2024 20:47
github-actions bot pushed a commit that referenced this pull request Aug 3, 2024
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
Copy link
Contributor

Awesome, thanks for taking this over!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support @typescript-eslint/utils v8
3 participants