-
-
Notifications
You must be signed in to change notification settings - Fork 28
ttsc --watch not working when upgrade typescript-transform-paths to 2.0.3 #78
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
Comments
Thanks for filing! I was working on a similar issue today and was able to work out what's going on. In your case, the short version is, the parent node isn't being set on our workaround node. There also doesn't seem to be a way that we can do it without messing up the output. You can read more on that if you're interested in this issue. Ultimately, we're going to have to wait on TS to fix the issue that we've been working around. The TS issue is in the roster for 4.2.0, and I'm keeping an eye on it. I'm going to leave this issue open also, however, so that when we have a fix, I'll be sure to test your watch-mode case specifically. |
@nonara Just checking if this is different from #72? You commented on a few days ago indicating that the issue there is fixed, but my understanding of the two issues is that they are the same. I'm experiencing the same issue above trying to use
|
@jonathanj It is different than #72, though somewhat related. It is a duplicate of #77 In v2.0.3, we were able to alleviate the issue for many cases, however, as these two issues have shown, there are still edge cases where it breaks. Background: It appears at the moment that our only option is to wait for a fix in TS. I've talked to some MSFT folks and gotten some detail on where to look, so I'll be looking at the TS source today to see if I can help expedite a fix for the TS issue. Once we can drop the workarounds, all of these issues will be settled. |
Just a heads up - this is fixed in v2.1.0. We implemented proper type elision. No more workarounds. If interested, more detail can be found in #81 |
@nonara Thanks |
Operating procedures:
ttsc --watch
cevek/ttypescript#92
The text was updated successfully, but these errors were encountered: