-
Notifications
You must be signed in to change notification settings - Fork 44
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
Adding zod validation #691
Conversation
|
Variables changedNo variables changed |
Design Token Diff
|
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.
Just a few suggestions from reviewing the visual regression snapshots. I am not smart enough to review all the cool Zod logic so you may want to find someone to review! 😄
}, | ||
"medium": { | ||
"$value": "calc(32/20)" | ||
"$value": 1.6, |
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.
Slight shift in this one in the snapshot test, but thats probably because calc may resolve differently.. but it depends on the browser so they should be the same? Anyhow its so tiny I'm not sure we need to worry about it.
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.
But why, this is exactly 1.6
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 don't know why, and the screenshots are taken with linux so I can't really test 😛 its so tiny, do you think we should just accept the small shift that might happen?
b57355b
to
cc20b97
Compare
The snaphots are failing because I transformed some |
Summary
List of notable changes:
What should reviewers focus on?
Steps to test: