-
-
Notifications
You must be signed in to change notification settings - Fork 46
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 partial syntax, source maps, removed Keyword
from AST and other CLI fixes
#107
Conversation
Keyword
from AST and other CLI fixesKeyword
from AST and other CLI fixes
Hey @jasikpark you know about fuzzing! Looking at a recent naive run it is showing a mismatch
I was a little confused at first but I think |
Really want to get ✅ on one of the fuzzing tests. While most of the time the errors are extreme edge cases, it would still be nice. Unfortunately they don't build on windows so having to do this trial and error remotely. There also seems to be a regression in parsing performance introduced in this PR :(. |
Merging, will open an issue for the fuzzing tests |
Keyword
from ASTAdded partial syntax
Added the ability for type annotations and expression to be missing in the source. This allows the following to be valid AST (under a parse option)
Which should allow type checking with invalid source which is vital to the LSP
Source map bindings
This adds source map generation using the positions in the output and source
Added printing arguments across new lines for function calling
Still to do