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

Question about a upgrade TS to version 3.9 #816

Closed
szagi3891 opened this issue May 13, 2020 · 7 comments
Closed

Question about a upgrade TS to version 3.9 #816

szagi3891 opened this issue May 13, 2020 · 7 comments

Comments

@szagi3891
Copy link

Can I ask for upgrading the TS version to 3.9?

The current version of morph does not want to work with the new TS.

Argument of type 'import("/home/grzegorz/Pulpit/Projekty/gpp-website3/node_modules/ts-morph/lib/ts-morph").Node<import("/home/grzegorz/Pulpit/Projekty/gpp-website3/node_modules/@ts-morph/common/node_modules/typescript/lib/typescript").Node>' is not assignable to parameter of type 'import("/home/grzegorz/Pulpit/Projekty/gpp-website3/node_modules/ts-morph/lib/ts-morph").Node<import("/home/grzegorz/Pulpit/Projekty/gpp-website3/node_modules/typescript/lib/typescript").Node>
@jasonkuhrt
Copy link

Thanks for the great project @dsherret. We're also awaiting it in graphql-nexus/nexus#859. I'll try to find time in the coming days to help with a pull-request.

@dsherret
Copy link
Owner

dsherret commented May 16, 2020

Sorry, missed this until now and upgrading wasn't a priority since there wasn't much changes in TS 3.9. I did have this done locally on my machine though with the RC version, so I just had to change it to 3.9.2.

Highly recommend using the ts export from ts-morph to ensure both your code and ts-morph are using the same compiler version so you don't run into trouble 🙂

import { ts } from "ts-morph";

I'll do a release for this soon.

@szagi3891
Copy link
Author

Thank you very much. This completely solves my problem :)

@jasonkuhrt
Copy link

jasonkuhrt commented May 22, 2020

Hey @dsherret Maybe I missed something but IIUC 3.9 isn't supported yet. Can we keep this issue open until it is? Will make things easier to track on my side, if you don't mind!

@dsherret
Copy link
Owner

@jasonkuhrt I just forgot to merge master into latest (done now). Forgot I was trying that branching strategy for this project. I just double checked on npm and it's specifying 3.9.2.

@jasonkuhrt
Copy link

Ah awesome! Thanks

@jasonkuhrt
Copy link

jasonkuhrt commented May 23, 2020

Just to follow up on my comments above, it turns out our issue was never related to ts-morph. If you're curious it looks like TS 3.9 has a regression. TS Playground link. TS issue.

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

No branches or pull requests

3 participants