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
[Merged by Bors] - feat: long overdue delaborators #7633
Conversation
Note that this takes advantage of the fact that `(x : α) → p x` notation is | ||
never used for propositions, so we can match on this result and rewrite it. -/ | ||
@[scoped delab forallE] | ||
def delabPi : Delab := whenPPOption Lean.getPPNotation do | ||
let stx ← delabForall | ||
def delabPi' : Delab := whenPPOption Lean.getPPNotation do |
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.
Why do we have two pi delaborators here?
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.
Because some people actually like (x : α) → p x
instead of Π (x : α), p x
so the nice one is scoped.
This looks good to me, modulo a plan to replace this code with something better eventually. There is no need to block on this now if people need these delaborators but I'm sure we can do better. bors r+ |
This brings to Mathlib assorted delaborators written by @kmill for various projects including Mathematics in Lean. They mostly fix regressions compared to Lean 3. Co-authored-by: Mario Carneiro <mcarneir@andrew.cmu.edu>
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
This brings to Mathlib assorted delaborators written by @kmill for various projects including Mathematics in Lean. They mostly fix regressions compared to Lean 3.