-
-
Notifications
You must be signed in to change notification settings - Fork 203
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
TypeScript <3.4
support
#336
Comments
Do nothing.
|
Is there a way for us to document our minimal TypeScript supported version? |
Not in a way that warns the user I think. We could note it in the readme, but we both know that will not help much. |
I've done some quick search and it does not seem like TypeScript supports specifying minimal TypeScript version. As you say I don't think we can assume users will always use the latest TypeScript version (major+minor+patch). New TypeScript versions are generally mostly compatible with older ones, but they also introduce new features that won't work on older versions. Should we add a comment on top of the I am also going to check if it's among the 3862 opened issues in the TypeScript repo, and post an issue there otherwise. If there is really no solution to that problem (which I am surprised of), I would consider creating a package to solve it. |
Sure
👍 |
Issue at microsoft/TypeScript#32166 Let's see what happens there first. |
The guys at TypeScript are saying that supporting a minimal TypeScript version should be specified by using: "typesVersions": {
">=3.4": { "*": ["index.d.ts"] }
} This should produce an error message for users of older TypeScript versions: However there seems to be a bug there. |
Just got burned by this. We cannot afford to upgrade to 3.4 as there are other changes that break our code. Wish there was some way to simply ignore that In my opinion as much as nicer it makes the code, would be really that bad to replace all Or at least the workaround microsoft/TypeScript#32166 (comment) sounds like another way. Having empty |
I would like @sindresorhus input on this. I think you meant |
No, I mean for example this line ... Line 116 in 071a815
The But yea, there is a need for a bunch of |
Yes sorry you're right. Still would need some feedback from @sindresorhus on this. |
Also requires updating TypeScript to v3.4+ (See: sindresorhus/execa#336)
Also requires updating TypeScript to v3.4+ (See: sindresorhus/execa#336)
Also requires updating TypeScript to v3.4+ (See: sindresorhus/execa#336)
Also requires updating TypeScript to v3.4+ (See: sindresorhus/execa#336)
* [now-cli] Update `execa` to v3.2.0 Also requires updating TypeScript to v3.4+ (See: sindresorhus/execa#336) * Fix tests: code -> exitCode * code -> exitCode * More .code to .exitCode * Update `@zeit/fun` to v0.10.3
TypeScript 3.4 is old now, so we can close this. |
The types do not work with TypeScript
3.3
(see #335) and below notably because of the changes to thereadonly
keyword brought by TypeScript3.4
. What should we do about it?typesVersions
field to declare that we only support>=3.4
index.d.ts
for lower versions, and use thetypesVersions
field to distinguish it.The text was updated successfully, but these errors were encountered: