Conversation
f125708 to
42ae19e
Compare
739e01f to
e7160b5
Compare
|
@lzy0505 This PR is ready to review when you have time. |
e7160b5 to
52e7a40
Compare
52e7a40 to
3d98cf7
Compare
lzy0505
left a comment
There was a problem hiding this comment.
Looks good! I pushed some docstrings and comments. Another feature that is nice to have is destructing the asserted hypothesis. Do you think you can also add this?
Indeed, this is a nice feature. Lean is a bit weird since it has |
Add the ihave : Q tactic that allows asserting new separation logic hypothesis Add ihave to tactics.md and update tactics.md Improve error message of iapply and add test Closes #116
b4b3c6f to
9ab8eb1
Compare
|
I added support for destructuring to |
|
Good to go. |
|
You actually can destruct with both Otherwise, I haven't looked through it closely, but I trust Zongyuan's review and am excited to play around with it! |
|
Interesting, I thought I tried it in other settings and there |
9ab8eb1 to
d3be0db
Compare
|
Are we merging this? |
|
I think we're all good to go I'll just do it |
Builds on #119
Add the ihave : Q tactic that allows asserting new separation logic
hypothesis
Closes #116