chore: refactor keploy codebase (#1684) #11
YaxhveerAnnotations
1 error and 9 warnings
|
lint
issues found
|
|
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
lint:
pkg/core/proxy/integrations/util/util.go#L34
var-naming: don't use leading k in Go names; func parameter kMin should be min (revive)
|
|
lint:
utils/utils.go#L389
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
pkg/core/app/docker/docker.go#L23
var-naming: don't use leading k in Go names; const kDefaultTimeoutForDockerQuery should be defaultTimeoutForDockerQuery (revive)
|
|
lint:
pkg/core/app/app.go#L485
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
pkg/core/app/app.go#L229
var-naming: don't use leading k in Go names; var kInspect should be inspect (revive)
|
|
lint:
pkg/core/hooks/hooks.go#L456
var-naming: don't use leading k in Go names; var kInode should be inode (revive)
|
|
lint:
pkg/core/hooks/kernelComm.go#L64
var-naming: don't use leading k in Go names; method parameter kPid should be pid (revive)
|
|
lint:
pkg/core/proxy/dns.go#L180
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|