-
Notifications
You must be signed in to change notification settings - Fork 0
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 supported typescript version #943
Conversation
To follow typescript-eslint's compatibility, after latest update in b2bcf66 Reference info: https://github.com/typescript-eslint/typescript-eslint/tree/v4.31.1#supported-typescript-version
73575e4
to
0a4319b
Compare
0a4319b
to
43f4f0a
Compare
yarn.lock
Outdated
"@typescript-eslint/experimental-utils@4.31.1": | ||
version "4.31.1" | ||
resolved "https://registry.yarnpkg.com/@typescript-eslint/experimental-utils/-/experimental-utils-4.31.1.tgz#0c900f832f270b88e13e51753647b02d08371ce5" | ||
integrity sha512-NtoPsqmcSsWty0mcL5nTZXMf7Ei0Xr2MT8jWjXMVgRK0/1qeQ2jZzLFUh4QtyJ4+/lPUyMw5cSfeeME+Zrtp9Q== | ||
dependencies: | ||
"@types/json-schema" "^7.0.7" | ||
"@typescript-eslint/scope-manager" "4.31.1" | ||
"@typescript-eslint/types" "4.31.1" | ||
"@typescript-eslint/typescript-estree" "4.31.1" | ||
eslint-scope "^5.1.1" | ||
eslint-utils "^3.0.0" | ||
|
||
"@typescript-eslint/experimental-utils@^4.0.1": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be deduplicated
Typescript's stdlib only has the Error type from the browser, when Node.js actually uses an Error type with more properties. To avoid type errors, we use a type assertion inspired from https://dev.to/jdbar/the-problem-with-handling-node-js-errors-in-typescript-and-the-workaround-m64 Note: had to add a `globals` comment, since Eslint's `no-undef` rule does not handle the Noje.js global types.
43f4f0a
to
dcd6ba2
Compare
Did a bit too much thinking for nothing on this one, it should be much simpler like this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you rerun yarn install
to regenerate yarn.lock
? Otherwise it looks all good!
dcd6ba2
to
fac1f96
Compare
Argh, not very used to this deduplicating schenanigans yet. Thanks! |
Why
We want to be able to use frolint with more recent versions of TS.
ref #876
To follow typescript-eslint's compatibility, after latest update in b2bcf66
Reference info:
https://github.com/typescript-eslint/typescript-eslint/tree/v4.31.1#supported-typescript-version
What
Update
typescript-eslint
to the latest version. Update thetypescript
dependency to the link above.Also fixed a few type errors in the code caused by the update.