-
Notifications
You must be signed in to change notification settings - Fork 15
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
Applying patch from #37 #38
base: master
Are you sure you want to change the base?
Conversation
Hi @zerkalica , is there anything else I can do to help get this complete? |
@zerkalica any chance of getting this PR merged in any time soon? Kind of a blocker for I imagine a few projects :) |
fwiw I published a patched version @ovos-media/ts-transform-paths - feel free to use it until this PR gets merged |
I'd also like to see this merged, it's a blocker for some of my projects! |
I think it is possible that this project is not maintained anymore (take a look at Pulse page). @falkenhawk provided a fork. I will be using that from now on. |
Use |
Changing to |
I switched to https://www.npmjs.com/package/typescript-transform-paths, just had to edit the "paths" tsconfig setting, otherwise works the same just replace this abandoned plugin with that one |
To fix #37
Can verify that the code in the patch mentioned in #37 resolved the issue with TypeScript 4.5.