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: zetaUnused option (option only) #6754

Merged
merged 1 commit into from
Jan 23, 2025
Merged

feat: zetaUnused option (option only) #6754

merged 1 commit into from
Jan 23, 2025

Conversation

nomeata
Copy link
Collaborator

@nomeata nomeata commented Jan 23, 2025

This PR adds the +zetaUnused option.

Implementation to follow after the stage0 update.

This PR adds the `+zetaUnused` option.

Implementation to follow after the stage0 update.
@nomeata nomeata requested a review from leodemoura as a code owner January 23, 2025 14:18
@nomeata nomeata added the will-merge-soon …unless someone speaks up label Jan 23, 2025
@nomeata nomeata enabled auto-merge January 23, 2025 14:18
@nomeata nomeata added the changelog-language Language features, tactics, and metaprograms label Jan 23, 2025
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc January 23, 2025 14:31 Inactive
@nomeata nomeata added this pull request to the merge queue Jan 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 Jan 23, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Jan 23, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Jan 23, 2025
Merged via the queue into master with commit 7e03920 Jan 23, 2025
20 of 22 checks passed
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Jan 23, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

@nomeata nomeata deleted the joachim/zeta-unused branch January 23, 2025 17:22
github-merge-queue bot pushed a commit that referenced this pull request Jan 26, 2025
This PR implements the `zetaUnused` simp and reduction option (added in
#6754).

True by default, and implied by `zeta`, this can be turned off to make
simp even more careful about preserving the expression structure,
including unused let and have expressions.

Breaking change: The `split` tactic no longer removes unused let and
have expressions as a side-effect, in rare cases this may break proofs.
`dsimp only` can be used to remove unused have and let expressions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builds-mathlib CI has verified that Mathlib builds against this PR changelog-language Language features, tactics, and metaprograms toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN will-merge-soon …unless someone speaks up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants