Skip to content

warn on unknown keys at the top level #965

Closed

Description

@asottile

We're now warning on unknown keys at the hook level, this should also happen at the top level

Activity

  1. yoavcaspi commented on May 7, 2019

    @yoavcaspi
    Contributor

    I wish to work on this issue.
    I'm pre-commit newbie.
    can you refer me to the warning on unknown keys at the hook level?

  2. asottile commented on May 7, 2019

    @asottile
    MemberAuthor

    sure! @mitzkia started some work on this in #980 and added a WarnAdditionalKeys helper to cfgv to make this easier to do -- but looks like they've been busy with other things :)

    the extraneous keys for the hook level is done later in the process here -- though for the top level and repository level this should probably get done in pre_commit/clientlib.py (on the two mapping types directly) via the new WarnAdditionalKeys

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions