-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat(hog): parser updates for lambdas #24575
Conversation
It looks like the code of |
The parser was released in the root PR with these same changes under the 1.0.38 version. This error can be ignored. |
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.
"yeah, looks like code"
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Problem
Splitting out work from #24369
Changes
x -> x
everywhere(x, y) -> { a(x); y; }
columnExpr
--> removes specialcolumnArg
fn(parameters)(arguments)
(params & args). Fix a parser bug in the existing code when params were there, but an empty list.asd[0].foo[1]()
)How did you test this code?
Added and updated tests