Skip to content

feat: different syntax for new clear_value tactic#8516

Merged
kmill merged 2 commits intoleanprover:masterfrom
kmill:kmill_clear_value_2
May 28, 2025
Merged

feat: different syntax for new clear_value tactic#8516
kmill merged 2 commits intoleanprover:masterfrom
kmill:kmill_clear_value_2

Conversation

@kmill
Copy link
Copy Markdown
Collaborator

@kmill kmill commented May 28, 2025

This PR is a followup to #8449 to refine the syntax of clear_value. The syntax for adding equality hypotheses before clearing values is now clear_value (h : x = _). Any expression definitionally equal to x can be used in place of the underscore.

This syntax was developed in a Zulip discussion.

@kmill kmill requested a review from kim-em as a code owner May 28, 2025 12:30
@kmill kmill added the changelog-language Language features and metaprograms label May 28, 2025
Copy link
Copy Markdown
Contributor

@sgraf812 sgraf812 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just found a typo; resisted the temptation to do a full code review on PTO :)

@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 May 28, 2025
ghost pushed a commit to leanprover-community/batteries that referenced this pull request May 28, 2025
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 28, 2025
@ghost ghost added the builds-mathlib CI has verified that Mathlib builds against this PR label May 28, 2025
@ghost
Copy link
Copy Markdown

ghost commented May 28, 2025

Mathlib CI status (docs):

  • ✅ Mathlib branch lean-pr-testing-8516 has successfully built against this PR. (2025-05-28 14:07:54) View Log
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 5814c1e757ced7d55e047edd062c1b9e155ae782 --onto 3af9ab64ed79a667fb8989f7a0c258b018aba371. You can force Mathlib CI using the force-mathlib-ci label. (2025-05-28 22:46:00)

@kmill kmill added this pull request to the merge queue May 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 28, 2025
@kmill kmill enabled auto-merge May 28, 2025 21:09
This PR is a followup to leanprover#8449 to refine the syntax of `clear_value`. Now the syntax for adding equality hypotheses before clearing values is `clear_value (h : x = _)`.

This syntax was developed in a [Zulip discussion](https://leanprover.zulipchat.com/#narrow/channel/270676-lean4/topic/.60clear_value.60.20syntax.20request.20for.20comments/near/520704290).
@kmill kmill force-pushed the kmill_clear_value_2 branch from 020a2c9 to 5d1268d Compare May 28, 2025 21:59
@kmill kmill added this pull request to the merge queue May 28, 2025
Merged via the queue into leanprover:master with commit 4dd8648 May 28, 2025
14 checks passed
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 and metaprograms 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