Skip to content

feat: add hint to unusedVariables linter#14259

Open
wkrozowski wants to merge 2 commits into
leanprover:masterfrom
wkrozowski:wojciech/unusedVariablesHint
Open

feat: add hint to unusedVariables linter#14259
wkrozowski wants to merge 2 commits into
leanprover:masterfrom
wkrozowski:wojciech/unusedVariablesHint

Conversation

@wkrozowski

Copy link
Copy Markdown
Contributor

This PR adds a hint to a unusedVariables linter, suggesting to rename the unreferenced name with an underscore.

@wkrozowski wkrozowski added changelog-language Language features and metaprograms force-mathlib-ci labels Jul 3, 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 3, 2026
@leanprover-bot leanprover-bot added the builds-manual CI has verified that the Lean Language Reference builds against this PR label Jul 3, 2026
@leanprover-bot

leanprover-bot commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ✅ Reference manual branch lean-pr-testing-14259 has successfully built against this PR. (2026-07-03 10:54:10) View Log
  • 🟡 Reference manual branch lean-pr-testing-14259 build against this PR didn't complete normally. (2026-07-03 10:54:36) View Log
  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2026-07-07 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2026-07-08 11:10:47)

@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 3, 2026
@mathlib-lean-pr-testing

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

Copy link
Copy Markdown

Mathlib CI status (docs):

@mathlib-lean-pr-testing mathlib-lean-pr-testing Bot added builds-mathlib CI has verified that Mathlib builds against this PR and removed breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan labels Jul 3, 2026
mathlib-nightly-testing Bot pushed a commit to leanprover-community/batteries that referenced this pull request Jul 7, 2026
@wkrozowski wkrozowski force-pushed the wojciech/unusedVariablesHint branch from e1744a2 to 84227d3 Compare July 8, 2026 10:40
@wkrozowski wkrozowski removed request for hargoniX and mhuisi July 8, 2026 10:40
@wkrozowski wkrozowski force-pushed the wojciech/unusedVariablesHint branch from 84227d3 to 629bcd9 Compare July 8, 2026 10:42
mathlib-nightly-testing Bot pushed a commit to leanprover-community/batteries that referenced this pull request Jul 8, 2026
mathlib-nightly-testing Bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Jul 8, 2026
@mathlib-lean-pr-testing mathlib-lean-pr-testing Bot added breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan builds-mathlib CI has verified that Mathlib builds against this PR and removed builds-mathlib CI has verified that Mathlib builds against this PR breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan labels Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-manual CI has verified that the Lean Language Reference builds against this PR builds-mathlib CI has verified that Mathlib builds against this PR changelog-language Language features and metaprograms force-mathlib-ci 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