You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Foreword: This relates to an issue that was already opened and subsequently closed (I believe incorrectly). As such I've opened a new issue because I'm not sure whether a comment on the old issue would be picked up. If I'm wrong, then sorry, my mistake, let me know and in future I'll do that.
Issue:
Originally issue #853 reported that just running through the default setup and attempting a lint operation would result in a warning about Typescript versions being unsupported. Example error message:
> gts lint
version: 22
=============
WARNING: You are currently running a version of TypeScript which is not officially supported by @typescript-eslint/typescript-estree.
You may find that it works just fine, or you may not.
SUPPORTED TYPESCRIPT VERSIONS: >=3.3.1 <5.2.0
YOUR TYPESCRIPT VERSION: 5.4.5
Please only submit bug reports when using the officially supported version.
This was closed "fixed" as a result of #859 but it really isn't.
Even just from a "common sense" point of view the fix doesn't line up:
We received an error that we were using a TS version too high for the typescript-eslint/typescript-estree dependency.
A "fix" was committed that bumped the TS version even higher.
Steps to reproduce:
Create a new empty folder
run npm i gts
run npx gts init to create a default template
run npx gts lint
Summary
Previous issue was marked closed even though the issue persisted and no further guidance was given. I'll be honest I'm not really sure that there is a direct fix for this. If it was a personal project I'd downgrade TS version to be in the bounds of the typescript-estree dependency. But I get that as this has already been rolled out for a while with a higher TS version you probably don't want to downgrade and risk messing with people's existing projects.
Perhaps some "official" guidance on whether this is an issue / whether people should be worried about it for newcomers in the comments of this issue might help.
Seeing it closed "fixed" in the original issue made me scratch my head for a minute and doubt whether I'd done something wrong / recheck my dependencies.
The text was updated successfully, but these errors were encountered:
You are correct. I think one of our commits got out of sync since we can't yet drop support for Node 16/18. I'll leave this reopened until those are done too.
Foreword: This relates to an issue that was already opened and subsequently closed (I believe incorrectly). As such I've opened a new issue because I'm not sure whether a comment on the old issue would be picked up. If I'm wrong, then sorry, my mistake, let me know and in future I'll do that.
Issue:
Originally issue #853 reported that just running through the default setup and attempting a lint operation would result in a warning about Typescript versions being unsupported. Example error message:
This was closed "fixed" as a result of #859 but it really isn't.
Even just from a "common sense" point of view the fix doesn't line up:
Steps to reproduce:
npm i gts
npx gts init
to create a default templatenpx gts lint
Summary
Previous issue was marked closed even though the issue persisted and no further guidance was given. I'll be honest I'm not really sure that there is a direct fix for this. If it was a personal project I'd downgrade TS version to be in the bounds of the typescript-estree dependency. But I get that as this has already been rolled out for a while with a higher TS version you probably don't want to downgrade and risk messing with people's existing projects.
Perhaps some "official" guidance on whether this is an issue / whether people should be worried about it for newcomers in the comments of this issue might help.
Seeing it closed "fixed" in the original issue made me scratch my head for a minute and doubt whether I'd done something wrong / recheck my dependencies.
The text was updated successfully, but these errors were encountered: