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.

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

feat: make coeFun delaborator respect pp.tagAppFns #6729

Merged
merged 2 commits into from
Feb 2, 2025

Conversation

kmill
Copy link
Collaborator

@kmill kmill commented Jan 21, 2025

This PR makes the pretty printer for .coeFun-tagged functions respect pp.tagAppFns. The effect is that in docgen, when an expression pretty prints as f x y z with f a coerced function, then if f is a constant it will be linkified.

This PR makes the pretty printer for `.coeFun`-tagged functions respect how constants are annotated when `pp.tagAppFns` is true. The effect is that in docgen, when an expression pretty prints as `f x y z`, where this is implicitly a function coercion, then if `f` is a constant it will be linkified.
@kmill kmill added the changelog-pp Pretty printing label Jan 21, 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 Jan 21, 2025
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Jan 21, 2025

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase eb30249b1118b5bea767354d5362f4afba97c82e --onto f9e904af5022a57adb6105cda65d1569449346c2. (2025-01-21 17:42:01)
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase eb30249b1118b5bea767354d5362f4afba97c82e --onto d68c2ce28bc11ec0271a97aa7a7316905f15a484. (2025-02-02 22:57:11)

@kmill kmill enabled auto-merge February 2, 2025 22:34
@kmill kmill added this pull request to the merge queue Feb 2, 2025
Merged via the queue into leanprover:master with commit 89d897a Feb 2, 2025
16 checks passed
luisacicolini pushed a commit to opencompl/lean4 that referenced this pull request Feb 24, 2025
This PR makes the pretty printer for `.coeFun`-tagged functions respect
`pp.tagAppFns`. The effect is that in docgen, when an expression pretty
prints as `f x y z` with `f` a coerced function, then if `f` is a
constant it will be linkified.
luisacicolini pushed a commit to opencompl/lean4 that referenced this pull request Feb 25, 2025
This PR makes the pretty printer for `.coeFun`-tagged functions respect
`pp.tagAppFns`. The effect is that in docgen, when an expression pretty
prints as `f x y z` with `f` a coerced function, then if `f` is a
constant it will be linkified.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

2 participants