Skip to content

feat: optionally let Sym.dsimp rewrite in instances - #14459

Merged
leodemoura merged 1 commit into
masterfrom
hbv/dsimp_instances
Jul 20, 2026
Merged

feat: optionally let Sym.dsimp rewrite in instances#14459
leodemoura merged 1 commit into
masterfrom
hbv/dsimp_instances

Conversation

@hargoniX

Copy link
Copy Markdown
Member

This PR adds an option for Sym.dsimp to rewrite in instances. This is usually not desirable as it can lead to non-standard instances. However, we might for example want to rewrite ground terms in instances to make more terms syntactically equal.

@hargoniX hargoniX added the changelog-tactics User facing tactics label Jul 20, 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 20, 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 33ae929879bfcec8c4bceadf3dbc91172b36365f --onto 25ba8c3d3bcb1dded7ff5a6f3b6044b0a7970198. You can force Mathlib CI using the force-mathlib-ci label. (2026-07-20 10:03:45)

@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 33ae929879bfcec8c4bceadf3dbc91172b36365f --onto 49ff95727f98d43984726b26742d17a1ceea9dd5. You can force reference manual CI using the force-manual-ci label. (2026-07-20 10:03:47)

@hargoniX
hargoniX force-pushed the hbv/dsimp_instances branch from f26fed0 to 9859906 Compare July 20, 2026 15:28
@hargoniX
hargoniX force-pushed the hbv/dsimp_instances branch from 9859906 to 9934283 Compare July 20, 2026 15:32
@leodemoura
leodemoura added this pull request to the merge queue Jul 20, 2026
Merged via the queue into master with commit 8e7d074 Jul 20, 2026
17 checks passed
@hargoniX
hargoniX deleted the hbv/dsimp_instances branch July 20, 2026 20:28
robsimmons pushed a commit that referenced this pull request Jul 29, 2026
This PR adds an option for `Sym.dsimp` to rewrite in instances. This is
usually not desirable as it can lead to non-standard instances. However,
we might for example want to rewrite ground terms in instances to make
more terms syntactically equal.
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.

3 participants