Skip to content

Commit

Permalink
[CI] Restore the exported documentation linter (#12312)
Browse files Browse the repository at this point in the history
* Restore default linters
  • Loading branch information
amenasria committed Jun 23, 2022
1 parent 6d44ca2 commit 3249f76
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .golangci.yml
Expand Up @@ -4,6 +4,8 @@ run:
- cmd/agent/app/listchecks.go # ignore unused command for now
- pkg/util/cloudproviders/cloudfoundry/bbscache_test.go # implements interface from imported package whose method names fail linting
- pkg/util/intern/string.go # TODO: fix govet 'unsafeptr' error
- pkg/security/rconfig/rconfig.go # TODO: fix revive error (comment on RCPolicyProvider.Close)
- pkg/security/probe/selftests/tester.go # TODO: fix revive error (comment on Start)
- pkg/serverless/trace/inferredspan/constants.go # TODO: fox revive exported const error

issues:
Expand All @@ -18,6 +20,9 @@ issues:
- "`RandomString|TimeNowNano|InitLogging|SetHostname` is unused" # 'deadcode' errors in test/benchmarks/util.go
- "Error return value of `io.WriteString` is not checked" # 'errcheck' errors in tools/dep_tree_resolver/go_deps.go
- "Error return value of `pem.Encode` is not checked" # 'errcheck' errors in test/integration/utils/certificates.go
- "exported: exported type EasyjsonTime|NewEasyjsonTime|EasyjsonTime.MarshalEasyJSON|EasyjsonTime.UnmarshalJSON should have comment or be unexported" # 'revive' errors in pkg/security/utils/json.go
- "exported: exported const Exec should have comment \\(or a comment on this block\\) or be unexported" # 'revive' errors in pkg/process/events/model/model_common.go
- "exported: exported const APIName should have comment \\(or a comment on this block\\) or be unexported" # 'revive' errors in pkg/serverless/trace/inferredspan/constants.go
- "unnecessary conversion" # 'unconvert' errors in test/integration/utils/certificates_test.go
- "Error return value of .((os\\.)?std(out|err)\\..*|.*Close|.*Flush|os\\.Remove(All)?|.*print(f|ln)?|os\\.(Un)?Setenv). is not checked"
- "`eventContext` is unused"
Expand Down

0 comments on commit 3249f76

Please sign in to comment.