-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Update @typescript-eslint
to v3
#10177
Comments
@G-Rath seems I'm unable to actually assign you, so consider this message that 😀 |
@SimenB I did the upgrade to run I'm happy to go through them and have a go at replacing them with a stricter type, but if you could keep an eye out for usage of that type in PRs and ask authors to use a stricter type that would save me some work :) (I don't expect there to be many, but did notice #10073, which I've put a comment on). |
Happy to take a PR adding |
Totally! Landing iteratively is the way to go - I'll probably go package by package at this point, as there's quite a few uses of |
@G-Rath now that TS 4 is out (and I'd love to use labeled tuples), any idea how close we are to being able to upgrade? |
@SimenB getting there slowly :/ |
I'd say this is done via #10437 |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
If possible without extra peer dep warnings, updating to
eslint@7
would be nice as wellThe text was updated successfully, but these errors were encountered: