Skip to content

Conversation

@grunweg
Copy link
Contributor

@grunweg grunweg commented Dec 23, 2025

This PR ensures that pretty-printing of unification hints inserts a space after |- resp. ⊢.

All uses in Lean core and mathlib add a space after the |- or ⊢ symbol; this makes the output match usage in practice.
This was discovered in leanprover-community/mathlib4#30658, adding a formatting linter using pretty-printing as initial guide.

All uses in Lean core and mathlib add a space after the |- or ⊢ symbol.
Make the pretty-printer produce matching output.
This was discovered in leanprover-community/mathlib4#30658, adding a
formatting linter using pretty-printing as initial guide.
@grunweg
Copy link
Contributor Author

grunweg commented Dec 23, 2025

awaiting-review

@github-actions github-actions bot added the awaiting-review Waiting for someone to review the PR label Dec 23, 2025
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Dec 23, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Mathlib CI can not be attempted yet, as the nightly-testing-2025-12-23 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-mathlib, Mathlib CI should run now. You can force Mathlib CI using the force-mathlib-ci label. (2025-12-23 11:40:15)

@leanprover-bot
Copy link
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2025-12-23 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2025-12-23 11:40:17)

@kim-em kim-em added the changelog-language Language features and metaprograms label Jan 8, 2026
@kim-em kim-em added this pull request to the merge queue Jan 8, 2026
Merged via the queue into leanprover:master with commit 2e649e1 Jan 8, 2026
24 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review Waiting for someone to review the PR changelog-language Language features and metaprograms toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants