Skip to content

Make --fast meaningful again #1810

Description

@bconway

Problem: When every linter claims it's fast, "fast" no longer has meaning:

$ golangci-lint linters|grep 'fast: true'|wc -l
      68

$ golangci-lint linters|grep 'fast: false'
unused (megacheck): Checks Go code for ... [fast: false, auto-fix: false]

I don't want to pick on anyone's work, but many of the linters are in fact quite slow. I wouldn't use many of them with IDE on-save functionality, anyway.

A couple ideas:

  • Get rid of the "fast" categorization and get rid of --fast. This is probably not a good proposal.
  • Use a quantitative metric to decide what is "fast." Not sure what this would look like or when/where it would be performed.

Thanks for the consideration.

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

    area: CLIRelated to CLIarea: configRelated to .golangci.yml and/or cli optionsenhancementNew feature or improvement

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions