Skip to content

Enable only non-deprecated linter with enable-all #1986

Closed
@sanposhiho

Description

@sanposhiho

Is your feature request related to a problem? Please describe.

Now, enable-all enable all linters including deprecated linter.

[kenseinakada] /Users/kenseinakada/workspace/website-server
(*;-;)? < golangci-lint version                           
golangci-lint has version v1.40.0 built from (unknown, mod sum: "h1:MFueiIIh9Ri5yWLRu9RkrS0nd2F+x67zC7ISQR2Hta4=") on (unknown)

~/workspace/website-server master

[kenseinakada] /Users/kenseinakada/workspace/website-server
(*'-') < golangci-lint run ./... --enable-all --no-config
WARN [runner] The linter 'scopelint' is deprecated (since v1.39.0) due to: The repository of the linter has been deprecated by the owner.  Replaced by exportloopref. 
WARN [runner] The linter 'interfacer' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.  
WARN [runner] The linter 'maligned' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.  Replaced by govet 'fieldalignment'. 

# result

Describe the solution you'd like

If we want to avoid deprecated linter, we need to disable them using -disable
It is proposed to change the specification of deprecated linter to not be enabled by enable-all.

Describe alternatives you've considered

add a disable-deprecated option to automatically disable all deprecated linter

Additional context
(Yeah, I know that there is a discussion on whether to deprecate the enable-all option in #1888 :))

Metadata

Metadata

Assignees

No one assigned

    Labels

    blockedNeed's direct action from maintainerenhancementNew feature or improvement

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions