Skip to content

chore: check binder types before extending the local context - #14633

Merged
leodemoura merged 1 commit into
masterfrom
kernel_check_binder_order
Aug 1, 2026
Merged

chore: check binder types before extending the local context#14633
leodemoura merged 1 commit into
masterfrom
kernel_check_binder_order

Conversation

@leodemoura

Copy link
Copy Markdown
Member

This PR makes infer_lambda and infer_let check a binder's type, and for let also its value, before adding the corresponding declaration to the local context, which is what infer_pi already did. No valid declaration changes behavior.

This PR makes `infer_lambda` and `infer_let` check a binder's type, and for `let` also its value,
before adding the corresponding declaration to the local context, which is what `infer_pi` already did.
No valid declaration changes behaviour.
@leodemoura
leodemoura enabled auto-merge August 1, 2026 19:20
@leodemoura
leodemoura added this pull request to the merge queue Aug 1, 2026
@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 Aug 1, 2026
@mathlib-lean-pr-testing

Copy link
Copy Markdown

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 393d38abb065d369a8a84b5a3da1f4da2e8517ad --onto 5b8d06c1764fa367a1e27b3e89f64667eac425af. You can force Mathlib CI using the force-mathlib-ci label. (2026-08-01 19:38:50)

@leanprover-bot

Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 393d38abb065d369a8a84b5a3da1f4da2e8517ad --onto 23393b959b33e3a8d15796b2397f8a04c315b9f4. You can force reference manual CI using the force-manual-ci label. (2026-08-01 19:38:51)

Merged via the queue into master with commit 5fa71c9 Aug 1, 2026
25 checks passed
Garmelon pushed a commit that referenced this pull request Aug 3, 2026
This PR makes `infer_lambda` and `infer_let` check a binder's type, and
for `let` also its value, before adding the corresponding declaration to
the local context, which is what `infer_pi` already did. No valid
declaration changes behavior.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-other 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