Commit 2020-04-02 19:25 d3b8622b
View on Github →fix(tactic/lint): simp_nf: do not ignore errors (#2266)
This PR fixes some bugs in the simp_nf linter.  Previously it ignored all errors (from failing tactics).  I've changed this so that errors from linters are handled centrally and reported as linter warnings.  The simp_is_conditional function was also broken.
As usual, new linters find new issues:
- Apparently Lean sometimes throws away simp lemmas. https://github.com/leanprover-community/lean/issues/163
- Some types define has_coebut have an incorrecthas_coe_to_fun, causing the simplifier to loop⇑f a = ⇑↑f a = ⇑f a. See the new library note: