-
Notifications
You must be signed in to change notification settings - Fork 39
feat(proof-lib/lean) core models #1747
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
base: main
Are you sure you want to change the base?
Conversation
This commit adds core-models for: - Options - Results - Default - Fn/FnOnce
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.
Great!
About the tests, lean type-checking is not tested on these tests right? So do you manually try that it works?
| Hax Lean Backend - Cryspen | ||
|
|
||
| Core-model for [https://doc.rust-lang.org/src/core/ops/function.rs.html]: | ||
| The Default trait for types with a default value. |
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.
This description is for another module
|
Hi, looks good to me! As Maxime mentioned, it would be nice to actually test the Lean type-checking. I did it manually and it worked for me. The name clash between hax's |
This PR leverages the recent work on Core Models to add functions for several modules of Core :
options,result,default,clone,Fn/FnOnce.Overview
proof-libs/lean, models are added. They are lightly adapted (mostly namespacing) from a normalcargo hax into leanof the core-models crate.tests/lean-core-models, add tests for all the functions added in the lean library. The extracted lean is well-typed.rust-engine/backends/lean.rsfix a bug (missing parenthesis)