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
Infer type schema #514
Infer type schema #514
Conversation
* Added type inference * Test cases with tsd * Allow manual inference
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not really convinced to add json-schema-to-ts
as dependency for this project.
However, I do agree we should provide some generic params for the input validation.
@climba03003 I agree. The implementation should work across json schema libraries. I detailed why a generic only extending object is better for cross library support here. #509 (comment) Putting the onus on the user to provide the correct type is a good trade off for flexibility. |
So if everyone is ok I can update the PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Nice. Thanks @DanieleFedeli I appreciate you |
Checklist
npm run test
andnpm run benchmark
and the Code of conduct
This PR is linked to #509 and aims to propagate schema type to the final function
This comment showed the outcome.