-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Trying to use https://github.com/wessberg/DI-compiler with ts-patch #120
Comments
Hello! If I'm reading that correctly, it looks like You may also be able to simply return Let me know if that helps! Edit: I just saw your issue. Did this work with ttypescript before, and if so, can you show me what the original code was that worked? Tsp should be fully compatible without needing to change anything. |
i was not able to get things to work with ttypescript because of the typescript version I am working with, i can only assume it did as the read-me calls it out https://github.com/wessberg/DI-compiler#usage-with-ttypescript. I will give your suggestions ago and report back. thanks for the reply |
i just saw that my example was incorrect, it is in a form where I was playing with things try to work out the issue. should be:
its in this form where I get the error
|
I appreciate the update. I recently fixed a bug in that same area. It's possible we need to also add array support there. I'll take a look at it. In the mean time, does returning |
BOOM! yes it worked. I had to do this:
|
i will see what i can do with getting an MR to fix the issue for you; I am still only new to TypeScript so will see. |
Array support added in |
I have been trying to get the transformer @wessberg/di-compiler to work but just end up with an error
I have logged out the object it is returning an object with a before property. I am not sure what the issue is
my transformer:
my compilerOptions:
I am not sure what to do to resolve the issue, I looked at breaking down what its doing try creating a program transformer but before I started this I thought I would raise an issue in case I am missing something.
Thanks
The text was updated successfully, but these errors were encountered: