Skip to content

Audit flake8 CI errors vs. warnings #2

Description

@byarbrough

#1 mentions that the flake8 in workflows/lint.yml may be overly permissive. This could result in bad code getting in.

  • evaluate error on F401 for unused imports
  • evaluate error on F841 for local variable assigned but unused

Note that pre-commit config already prohibits both of these.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions