-
Notifications
You must be signed in to change notification settings - Fork 9
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
Validation #7
Comments
This comment has been minimized.
This comment has been minimized.
Do you consider supporting type-checking when validating? By type-checking, I mean: The following Web IDL is valid:
… whilst the following is invalid:
… because the I've a very naive algorithm that works with the AST only. If this is something required, I can improve the code and open a PR. |
Yes, validation would do type checking. I'm not quite ready for a PR for this yet though, I need to think about it a bit more and talk with the folks writing the standards and things like that to make sure it helps push that side of things forward. |
OK. Please ping me to keep me in the discussions. |
Should be able to take an AST and some extra wasm typing context (probably expressed as a trait) and validate that the bindings described by the AST are well-formed.
The text was updated successfully, but these errors were encountered: