8000 fix: update `Parser.Term.letIdDeclNoBinders` to use new `letIdDecl` format by kmill · Pull Request #8929 · leanprover/lean4 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: update Parser.Term.letIdDeclNoBinders to use new letIdDecl format #8929

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
merged 1 commit into from
Jun 22, 2025

Conversation

kmill
Copy link
Collaborator
@kmill kmill commented Jun 22, 2025

This PR is a followup to #8914, fixing an oversight where letIdDeclBinders is was not updated with the new format. This relies on some bootstrapping code to stay in place, but we do bootstrap cleanup that is currently possible.

…ormat

This PR is a followup to leanprover#8914, fixing an oversight where `letIdDeclBinders` is was not updated with the new format. This just means that we will need another stage0 update and cleanup. This PR does stage0 cleanup that is possible at this moment.
@kmill < 8000 a class="author Link--primary text-bold" data-hovercard-type="user" data-hovercard-url="/users/kmill/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="/kmill">kmill added the changelog-no Do not include this PR in the release changelog label Jun 22, 2025
@kmill kmill enabled auto-merge June 22, 2025 19:18
@kmill kmill added this pull request to the merge queue Jun 22, 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 Jun 22, 2025
@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 16c918a6522dda7855264b47e7ce8b97db97a0a7 --onto db499e96aac8ad654c8ed5ab40c4e6885d38c9a1. You can force Mathlib CI using the force-mathlib-ci label. (2025-06-22 19:43:42)

Merged via the queue into leanprover:master with commit f4f664e Jun 22, 2025
19 checks passed
kmill added a commit to kmill/lean4 that referenced this pull request Jun 22, 2025
This PR finishes post-stage0-cleanup after leanprover#8914 and leanprover#8929. Also:
- adds configuration options for `haveI` and `letI` terms.
- adds `letConfig` parser alias
github-merge-queue bot pushed a commit that referenced this pull request Jun 22, 2025
< 8000 a href="/leanprover/lean4/commit/bb0132e4b32e116627fe8b94b4d0025dd4b69801" class="Link--secondary">bb0132e
This PR finishes post-stage0-cleanup after #8914 and #8929. Also:
- adds configuration options for `haveI` and `letI` terms.
- adds `letConfig` parser alias
wkrozowski pushed a commit to wkrozowski/lean4 that referenced this pull request Jun 24, 2025
…ormat (leanprover#8929)

This PR is a followup to leanprover#8914, fixing an oversight where
`letIdDeclBinders` is was not updated with the new format. This relies
on some bootstrapping code to stay in place, but we do bootstrap cleanup
that is currently possible.
wkrozowski pushed a commit to wkrozowski/lean4 that referenced this pull request Jun 24, 2025
This PR finishes post-stage0-cleanup after leanprover#8914 and leanprover#8929. Also:
- adds configuration options for `haveI` and `letI` terms.
- adds `letConfig` parser alias
nomeata added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-no Do not include this PR in the release changelog 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
0