- Notifications
You must be signed in to change notification settings - Fork109
feat(typecheck): also replace longer type defaults#541
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| // all values of the enums are longer than `NULL`, so we use `NULL` instead | ||
| "select 0 + 0 + 0 + 0 + 0 + NULL " | ||
| "select 0 + 0 + 0 + 0 + 0 + 'critical'" |
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.
not sure if the change fromNULL to'critical' is desired?
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.
yes, makes sense after your change
Uh oh!
There was an error while loading.Please reload this page.
psteinroe left a comment
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.
Beautiful!
Uh oh!
There was an error while loading.Please reload this page.
| // all values of the enums are longer than `NULL`, so we use `NULL` instead | ||
| "select 0 + 0 + 0 + 0 + 0 + NULL " | ||
| "select 0 + 0 + 0 + 0 + 0 + 'critical'" |
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.
yes, makes sense after your change
fcd4720 intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Before, we would only show type errors if the length of the default-type-value was <= than the length of the param-name it belonged to.
Now we have an API that tracks positions in text replacements even if they grow the original text, so we can show such errors: