Skip to content

Commit 950ff56

Browse files
committed
chore(MathlibTest/grind/lint): improve documentation (leanprover-community#32935)
This PR updates the grind_lint test file to: - Explain why `Path.symm_apply` and `Set.Icc.convexCombo_symm` are skipped (they each instantiate 24 lemmas, which seems reasonable) - Improve the instructions for what to do when the test fails 🤖 Prepared with Claude Code
1 parent c98df61 commit 950ff56

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

MathlibTest/grind/lint.lean

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
import Mathlib
22

3-
#adaptation_note
4-
/--
5-
FIXME: please investigate these lemmas, and add guard conditions as needed.
6-
(These appeared at nightly-2025-12-13.)
7-
--/
3+
-- These each instantiate 24 further lemmas (pretty much the same ones), but they seem reasonable.
4+
-- We'll make an exception for this one,
5+
-- but if we accumulate too many we can raise the threshold below.
86
#grind_lint skip Path.symm_apply
97
#grind_lint skip Set.Icc.convexCombo_symm
108

119
-- This check verifies that `grind` annotations in Mathlib do not trigger run-away instantiations.
12-
-- If this test fails, please modify newly introduced `grind` annotations to use the
10+
-- If this test fails, please follow the "Try this:" suggestions
11+
-- which will explain the excessive instantations.
12+
-- Please modify newly introduced `grind` annotations to use the
1313
-- `grind_pattern ... where ...` syntax to add side conditions that will prevent the run-away.
1414
-- See https://lean-lang.org/doc/reference/latest/The--grind--tactic/E___matching/ for details.
1515
#grind_lint check (min := 20) in module Mathlib

0 commit comments

Comments
 (0)