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

Support ESLint 8 #728

Closed
2 tasks done
sQVe opened this issue Nov 7, 2021 · 9 comments · Fixed by #759
Closed
2 tasks done

Support ESLint 8 #728

sQVe opened this issue Nov 7, 2021 · 9 comments · Fixed by #759

Comments

@sQVe
Copy link

sQVe commented Nov 7, 2021

⭐ Support ESLint 8.

This is a tracking issue for supporting eslint@8. It's currently being blocked by:


New eslint rules with typescript-eslint equivalents:

New rules worth considering:


Refs:

@sQVe
Copy link
Author

sQVe commented Nov 7, 2021

I didn't have time going through all entries of what typescript-eslint actually recommends in their recommended config. My gut tells me that it might be worth giving it a glance though - there might be some useful rules that we've missed.

@meyfa
Copy link

meyfa commented Jan 9, 2022

standard/eslint-config-standard#193 has been merged. What does the process look like for this issue here, and is there some way to help?

@xCykrix
Copy link

xCykrix commented Jan 10, 2022

Would definitely like some news on getting this merged so I can update my toolchain. Is there anything that could be assisted with to get this rolling @sQVe ?

@sQVe
Copy link
Author

sQVe commented Jan 11, 2022

@xCykrix The first thing we would need to resolve is to update the dependencies for eslint@8. After that I feel think it would be great to have some consideration on the new rules that we could add.

@theoludwig
Copy link
Contributor

We wil add support for ESLint v8, when eslint-config-standard v17 has been released stable.

@sQVe
Copy link
Author

sQVe commented Jan 12, 2022

@divlo Until then it might make sense to have a discussion about the rules that applies only for TypeScript land?

@theoludwig
Copy link
Contributor

Sure we can! @sQVe

Maybe in different issues to make it more maintainable.

@ghost
Copy link

ghost commented Jan 17, 2022

@sQVe maybe you could make a next release that includes the next version of eslint-config-standard? So we could start testing it out and reporting any issues 🙏

@linonetwo
Copy link

When npm i -D eslint-config-standard@next

Could not resolve dependency:
npm ERR! peer @typescript-eslint/eslint-plugin@"^4.0.1" from [email protected]

this means eslint-config-standard-with-typescript need to update deps to come up with eslint-config-standard@next

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

Successfully merging a pull request may close this issue.

5 participants