Skip to content

feat: validate [grind homo] theorems - #14452

Merged
leodemoura merged 2 commits into
masterfrom
grind_homo_validate
Jul 19, 2026
Merged

feat: validate [grind homo] theorems#14452
leodemoura merged 2 commits into
masterfrom
grind_homo_validate

Conversation

@leodemoura

@leodemoura leodemoura commented Jul 19, 2026

Copy link
Copy Markdown
Member

This PR rejects [grind homo] theorems that are conditional rewriting rules. Conditional theorems are rejected with an error pointing to the E-matching attributes. The reset_grind_attrs% command now also clears the [grind homo] and [grind homo_pred] extensions.

🤖 Generated with Claude Code

This PR rejects `[grind homo]` theorems that cannot be applied as rewrite rules: instance-implicit parameters may be omitted from the left-hand side (they are synthesized during rewriting), but any other parameter must be determined by it. Conditional theorems are rejected with an error pointing to the E-matching attributes. The `reset_grind_attrs%` command now also clears the `[grind homo]` and `[grind homo_pred]` extensions.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@leodemoura leodemoura added the changelog-tactics User facing tactics label Jul 19, 2026
@leodemoura
leodemoura requested a review from kim-em as a code owner July 19, 2026 18:38
@leodemoura leodemoura added the changelog-tactics User facing tactics label Jul 19, 2026
@leodemoura
leodemoura enabled auto-merge July 19, 2026 18:40
@leodemoura
leodemoura added this pull request to the merge queue Jul 19, 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 Jul 19, 2026
@mathlib-lean-pr-testing

mathlib-lean-pr-testing Bot commented Jul 19, 2026

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 2cf39b81995e4c97c08bb5d54adae45bb8bb0f62 --onto 79bec6d064d8672cb10864cad3d9fd054b6917ae. You can force Mathlib CI using the force-mathlib-ci label. (2026-07-19 19:07:04)
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 1bc7a6154f80d79a260a1762b7634e7b4808eb8a --onto 79bec6d064d8672cb10864cad3d9fd054b6917ae. You can force Mathlib CI using the force-mathlib-ci label. (2026-07-19 19:46:02)

@leanprover-bot

leanprover-bot commented Jul 19, 2026

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 2cf39b81995e4c97c08bb5d54adae45bb8bb0f62 --onto 49ff95727f98d43984726b26742d17a1ceea9dd5. You can force reference manual CI using the force-manual-ci label. (2026-07-19 19:07:06)
  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 1bc7a6154f80d79a260a1762b7634e7b4808eb8a --onto 49ff95727f98d43984726b26742d17a1ceea9dd5. You can force reference manual CI using the force-manual-ci label. (2026-07-19 19:46:04)

@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Jul 19, 2026
@leodemoura
leodemoura enabled auto-merge July 19, 2026 19:11
@leodemoura
leodemoura added this pull request to the merge queue Jul 19, 2026
Merged via the queue into master with commit 263276d Jul 19, 2026
17 checks passed
robsimmons pushed a commit that referenced this pull request Jul 29, 2026
This PR rejects `[grind homo]` theorems that are conditional rewriting
rules. Conditional theorems are rejected with an error pointing to the
E-matching attributes. The `reset_grind_attrs%` command now also clears
the `[grind homo]` and `[grind homo_pred]` extensions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-tactics User facing tactics 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