Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: make sure #check id heeds pp.raw #6181

Merged
merged 2 commits into from
Nov 23, 2024
Merged

fix: make sure #check id heeds pp.raw #6181

merged 2 commits into from
Nov 23, 2024

Conversation

kmill
Copy link
Collaborator

@kmill kmill commented Nov 23, 2024

This PR fixes a bug where the signature pretty printer would ignore the current setting of pp.raw. This fixes an issue where #check ident would not heed pp.raw. Closes #6090.

@kmill
fix: make sure #check id heeds pp.raw
ed0c59b 
This PR fixes a bug where the signature pretty printer would ignore the current setting of `pp.raw`. This fixes an issue where `#check ident` would not heed `pp.raw`. Closes leanprover#6090.
@kmill kmill added the changelog-pp Pretty printing label Nov 23, 2024
@kmill kmill enabled auto-merge November 23, 2024 00:17
@kmill kmill added this pull request to the merge queue Nov 23, 2024
@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 Nov 23, 2024
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 5145030ff4bed884213f584f8b2402be82e96d45 --onto 6202461a21d2636129cb8950cd9b6549ccf4b185. (2024-11-23 00:43:49)

Merged via the queue into leanprover:master with commit ba3f2b3 Nov 23, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
changelog-pp Pretty printing toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

pp.raw not affecting signature printing
2 participants
@kmill @leanprover-community-bot