Refactor cmd/rigtest into a real test suite in test/rig_test.go and fix found bugs #249
kke
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
10 errors and 10 warnings
|
Lint:
ssh.go#L556
cognitive complexity 32 of func `(*SSH).Exec` is high (> 30) (gocognit)
|
|
Lint:
pkg/rigfs/types.go#L32
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
Lint:
pkg/rigfs/winfile.go#L181
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
Lint:
pkg/rigfs/winfileinfo.go#L10
File is not `gofumpt`-ed (gofumpt)
|
|
Lint:
pkg/rigfs/winfsys.go#L16
File is not `gofumpt`-ed (gofumpt)
|
|
Lint:
exec/exec.go#L100
`if o.LogCommand` has complex nested blocks (complexity: 7) (nestif)
|
|
Lint:
pkg/rigfs/winfile.go#L211
calculated cyclomatic complexity for function command is 15, max is 12 (cyclop)
|
|
Lint:
pkg/rigfs/posixfsys.go#L253
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"file %s is not open for reading\", f.path)" (goerr113)
|
|
Lint:
pkg/rigfs/winfile.go#L205
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"remote error: %s\", resp.Err)" (goerr113)
|
|
Lint:
pkg/rigfs/winfile.go#L238
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"rigrcp exited\")" (goerr113)
|
|
Lint:
pkg/rigfs/withname.go#L6
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
|
|
Lint:
pkg/rigfs/withname.go#L7
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
|
|
Lint:
pkg/rigfs/withname.go#L8
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
|
|
Lint:
pkg/rigfs/windir.go#L25
unused-parameter: parameter 'p' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Lint:
pkg/rigfs/windir.go#L29
unused-parameter: parameter 'pos' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Lint:
pkg/rigfs/withname.go#L9
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
|
|
Lint:
pkg/rigfs/withname.go#L10
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
|
|
Lint:
pkg/rigfs/windir.go#L33
unused-parameter: parameter 'p' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Lint:
pkg/rigfs/withname.go#L11
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
|
|
Lint:
pkg/rigfs/windir.go#L45
unused-parameter: parameter 'src' seems to be unused, consider removing or renaming it as _ (revive)
|