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
feat: add zod validation #369
Conversation
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 like that this doesn't affect end-users as much - but I think the zod types can be abstracted away more.
Happy to accept it as an improvement, but I don't necessarily see major benefit to loose coupling. Zod is a leading choice for this purpose. |
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'm working on a hopefully less invasive implementation of this, that doesn't affect the existing types or functionality.
This PR most likely supersedes #367