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
Fix generic #41
Fix generic #41
Conversation
Can you add a failing test case that passes with your changes? I don't write TypeScript and rely on test coverage for details like this. |
Sure! Can you enable the tests for branches as well? So I can see whether they pass here? |
Oh and re: writing new tests, it seems like the tests are actually wrong, so I will just fix them :) |
One thing that I would like to do (just as a fun project) is to add proper recursive types using the new Template Literal Types in TypeScript 4.1.2. There is a microsoft/TypeScript#40336 (comment) So then TypeScript would know exactly what properties are on the final object. |
@bendrucker tests and types are fixed (in |
Great work, thank you! |
Glad to help, thanks for the merge! |
Fixes implementation of #31