Skip to content

fix defsWithUnderscore linter #1147

@zhikaip

Description

@zhikaip

.lake\packages\mathlib\Mathlib\Tactic\Linter\Style.lean introduces a new defsWithUnderscore linter,

The most problematic issue is informal_lemma which elaborates to a def, probably need to include @[nolint defsWithUnderscore]

_physlib-suffixed instances are also flagged, the linter identifies _mathlib ones but not physlib

there's also some meta declarations e.g. semiformal_result, Sorryful_attr, Pseudo_attr

the linter exempts _1 and _2 but not _3 and _4 in Physlib.Mathematics.DataStructures.FourTree.Basic

linter turned off for now but should be fixed

Originally posted by @zhikaip in #1130 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions