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
chore(shadertools): Fully eliminate implicit typings #1619
Conversation
584506c
to
f9bc122
Compare
f9bc122
to
93e7746
Compare
e4f986b
to
a805125
Compare
}; | ||
|
||
const TYPE_DEFINITIONS = { | ||
const TYPE_DEFINITIONS: Record<string, {validate: (value: unknown, propType: PropType) => boolean}> = { |
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 think this would be better called VALIDATOR
or similar
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.
That is a good idea, addressed in #1679
const propTypes: Record<string, PropType> = {}; | ||
|
||
|
||
export function parsePropTypes(propDefs: Record<string, PropType>): Record<string, PropDef> { |
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 find the types and parameter naming confusing. It's not clear why you chose to swap the names of the types but kept the parameter names.
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.
Good catch. Overhauled in same PR #1679
For #1402
Background
Change List
noImplicitAny
in shadertools module