Skip to content

Commit

Permalink
Hygiene: enable additional linters. (#735)
Browse files Browse the repository at this point in the history
Enabled:
- containedctx
- dogsled
- duple
- errchkjson
- errorlint
- errname
- exhaustive
- goconst
- ireturn
- staticcheck
- unparam
- wrapcheck

There are no expected functional changes in this PR.

Context: tektoncd/pipeline#5899

/kind cleanup

- Submitter Checklist

- [N/A] Has [Docs](https:/tektoncd/community/blob/main/standards.md#docs) included if any changes are user facing
- [N/A] Has [Tests](https:/tektoncd/community/blob/main/standards.md#tests) included if any functionality added or changed
- [x] Follows the [commit message standard](https:/tektoncd/community/blob/main/standards.md#commits)
- [x] Meets the [Tekton contributor standards](https:/tektoncd/community/blob/main/standards.md) (including functionality, content, code)
- [x] Has a kind label. You can add one by adding a comment on this PR that contains `/kind <type>`. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
- [N/A] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
- [N/A] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

```release-note
NONE
```
  • Loading branch information
bendory authored Mar 7, 2023
1 parent 35651e8 commit db09b27
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions .golang-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,25 +3,37 @@ linters-settings:
linters:
enable:
- bodyclose
- containedctx
- decorder
- depguard
- dogsled
- dupl
- dupword
- errcheck
- errchkjson
- errname
- errorlint
- exhaustive
- goconst
- gocritic
- gofmt
- goimports
- gomodguard
- gosec
- gosimple
- govet
- ireturn
- misspell
- revive
- staticheck
- thelper
- typecheck
- unconvert
- unparam
- unused
- usestdlibvars
- whitespace
- wrapcheck
# Enabling presets means that new linters that we automatically adopt new
# linters that augment a preset. This also opts us in for replacement linters
# when a linter is deprecated.
Expand Down

0 comments on commit db09b27

Please sign in to comment.