Skip to content

Commit

Permalink
Address golangci-lint deprecation warnings, enable some more linters (#…
Browse files Browse the repository at this point in the history
…2152)

* Address golangci-lint linter deprecation warnings

1.59.0 outputs:

WARN [lintersdb] The name "gas" is deprecated. The linter has been renamed to: gosec.
WARN [lintersdb] The linter named "megacheck" is deprecated. It has been split into: gosimple, staticcheck, unused.

* Enable some more linters, address finding
  • Loading branch information
scop authored Jun 1, 2024
1 parent 8003b74 commit e94f6d0
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
12 changes: 5 additions & 7 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,28 +26,26 @@ linters:
- errcheck
#- exhaustive
#- funlen
- gas
#- gochecknoinits
- goconst
- gocritic
#- gocyclo
#- gofmt
- gofmt
- goimports
#- gomnd
#- goprintffuncname
#- gosec
- gosec
- gosimple
- govet
- ineffassign
#- lll
- megacheck
#- misspell
- misspell
#- nakedret
#- noctx
#- nolintlint
- nolintlint
#- rowserrcheck
#- scopelint
#- staticcheck
- staticcheck
#- structcheck ! deprecated since v1.49.0; replaced by 'unused'
- stylecheck
#- typecheck
Expand Down
1 change: 0 additions & 1 deletion command.go
Original file line number Diff line number Diff line change
Expand Up @@ -1460,7 +1460,6 @@ func (c *Command) UseLine() string {

// DebugFlags used to determine which flags have been assigned to which commands
// and which persist.
// nolint:goconst
func (c *Command) DebugFlags() {
c.Println("DebugFlags called on", c.Name())
var debugflags func(*Command)
Expand Down

0 comments on commit e94f6d0

Please sign in to comment.