-
Notifications
You must be signed in to change notification settings - Fork 138
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
Support for lambdas :) #2465
Support for lambdas :) #2465
Conversation
Some examples |
That is amazing!!!! Though some tests seems to fail :'( |
Yeah is just the usual suspect |
Added some tests and fixed the issue with |
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.
Thanks @AlecsFerra! I make one request for documentation below.
If there are some limitations to keep in mind, it may be worth mentioning them in the user documentation.
AFAIK there are no limitations |
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.
Thanks for the comments!
Co-authored-by: Niki Vazou <[email protected]>
Co-authored-by: Facundo Domínguez <[email protected]>
654e269
to
293f739
Compare
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.
LGTM!
Its breaking two tests related to typeclasses, both caused by the same file that breaks elaboration.
Companion PR in LF: ucsd-progsys/liquid-fixpoint#725