-
Notifications
You must be signed in to change notification settings - Fork 62
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
Add caketype
#1029
base: master
Are you sure you want to change the base?
Add caketype
#1029
Conversation
Thank you for your submission! There was a minor issue - we used pinned package versions. I've fixed it thru GitHub's suggestion mechanism. |
Hm, looks like build is failing with:
|
Thanks for looking into this! I'm guessing those errors occur because If I configure VS Code to use the workspace TypeScript version and open I don't currently plan to support TypeScript 4.6 in caketype either. How do you think we should proceed? Sorry about that 😓 |
I used this commit as a template, and tested it locally with
npm run start run zod caketype
.caketype
requires TypeScript 4.7, but the tests seem to run with TypeScript 4.6, which was causing type errors. However, setting the type of the schema object toany
suppresses these. I hope that's okay for the purposes of runtime benchmarking 😅Closes #1028.