feat: add config for CI linting and code quality improvements #100
djach7ci.yml
on: pull_request
Build and test makefile
31s
Build and test on Fedora latest
52s
version-release
0s
Annotations
11 errors
|
Build and test makefile:
pkg/tar-diff/analysis.go#L283
redefines-builtin-id: redefinition of the built-in function new (revive)
|
|
Build and test makefile:
pkg/tar-diff/analysis.go#L261
superfluous-else: if block ends with a break statement, so drop this else and outdent its block (revive)
|
|
Build and test makefile:
pkg/tar-diff/analysis.go#L220
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
Build and test makefile:
pkg/tar-diff/analysis.go#L147
superfluous-else: if block ends with a break statement, so drop this else and outdent its block (revive)
|
|
Build and test makefile:
pkg/common/version.go#L1
var-naming: avoid meaningless package names (revive)
|
|
Build and test makefile:
pkg/tar-diff/bsdiff.go#L39
cyclomatic complexity 32 of func `bsdiff` is high (> 30) (gocyclo)
|
|
Build and test makefile:
pkg/tar-diff/diff.go#L160
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
Build and test makefile:
pkg/tar-diff/bsdiff.go#L325
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
Build and test makefile:
cmd/tar-patch/main.go#L49
exitAfterDefer: log.Fatalf will exit, and `defer func(){...}(...)` will not run (gocritic)
|
|
Build and test makefile:
cmd/tar-diff/main.go#L56
exitAfterDefer: log.Fatalf will exit, and `defer func(){...}(...)` will not run (gocritic)
|
|
Build and test on Fedora latest
Process completed with exit code 2.
|