Add tactic `learn_hyp`, fixes #73
All threads resolved!
All threads resolved!
This is another part of !244 (closed) and fixes #73 (closed).
Edited by Robbert Krebbers
Merge request reports
Activity
added 3 commits
-
9eb7bd0f...e8ec0e0e - 2 commits from branch
master
- 8cb81510 - Add [learn_hyp_named] and [learn_hyp], fixes #73 (closed)
-
9eb7bd0f...e8ec0e0e - 2 commits from branch
- Resolved by Robbert Krebbers
Thanks for the MR.
- I think
P
is a confusing argument name oflearn_hyp
. The argument is a proof of a proposition, not a proposition itself. - Can we turn these things into
Tactic Notation
, so that we can saylearn_hyp P as H
- Can you write matches in the way we usually write them? So, a
|
on each line, don't use[ .. ]
. - The
match type of P with
can simply be alet X := type_of P
?
- I think
added 1 commit
- d9fe39d5 - Add [learn_hyp_named] and [learn_hyp], fixes #73 (closed)
added 8 commits
-
d9fe39d5...77cb2b8a - 7 commits from branch
master
- 56a9fc6b - Add [learn_hyp_named] and [learn_hyp], fixes #73 (closed)
-
d9fe39d5...77cb2b8a - 7 commits from branch
changed title from Add [learn_hyp_named] and [learn_hyp], fixes #73 (closed) to Add {+tactic
+}learn_hyp{+
+}, fixes #73 (closed)@jung Any opinion, for example on the name of the tactic?
Ok, merging! Thanks @msammler
mentioned in commit 59911631
Please register or sign in to reply