Skip to content

feat: make tactic set_option incremental - #14397

Merged
Kha merged 1 commit into
masterfrom
set-option-tactic-incremental
Jul 15, 2026
Merged

feat: make tactic set_option incremental#14397
Kha merged 1 commit into
masterfrom
set-option-tactic-incremental

Conversation

@Kha

@Kha Kha commented Jul 15, 2026

Copy link
Copy Markdown
Member

This PR makes the set_option ... in tactic support incremental elaboration, so edits inside its tactic block reuse the results of unchanged leading tactics instead of re-running the whole block.

This PR makes the `set_option ... in` tactic support incremental elaboration, so edits inside its tactic block reuse the results of unchanged leading tactics instead of re-running the whole block.

Reuse is narrowed to the body tactic sequence, so changing the option name or value disables reuse. Incrementality is disabled for `set_option diagnostics in` as partial reuse would make the diagnostic report emitted at the end of the tactic incomplete.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Kha
Kha requested a review from kim-em as a code owner July 15, 2026 11:42
@Kha Kha added the changelog-language Language features and metaprograms label Jul 15, 2026
@Kha
Kha enabled auto-merge July 15, 2026 11:42
@Kha
Kha added this pull request to the merge queue Jul 15, 2026
@github-actions github-actions Bot added toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN labels Jul 15, 2026
@leanprover-bot leanprover-bot added the builds-manual CI has verified that the Lean Language Reference builds against this PR label Jul 15, 2026
@leanprover-bot

leanprover-bot commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Reference manual CI status:

Merged via the queue into master with commit 47212d8 Jul 15, 2026
32 checks passed
@mathlib-lean-pr-testing mathlib-lean-pr-testing Bot added the breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan label Jul 15, 2026
@mathlib-lean-pr-testing

Copy link
Copy Markdown

Mathlib CI status (docs):

@Kha
Kha deleted the set-option-tactic-incremental branch July 15, 2026 15:59
wrenna-robson pushed a commit to wrenna-robson/lean4 that referenced this pull request Jul 17, 2026
This PR makes the `set_option ... in` tactic support incremental
elaboration, so edits inside its tactic block reuse the results of
unchanged leading tactics instead of re-running the whole block.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
robsimmons pushed a commit that referenced this pull request Jul 29, 2026
This PR makes the `set_option ... in` tactic support incremental
elaboration, so edits inside its tactic block reuse the results of
unchanged leading tactics instead of re-running the whole block.

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

breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan builds-manual CI has verified that the Lean Language Reference builds against this PR changelog-language Language features and metaprograms mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN 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