build(deps): bump morphy2k/revive-action from 2.5.3 to 2.5.4 #663
Annotations
7 errors and 10 warnings
Test on go 1.20 and ubuntu-latest:
envutil/envutil_test.go#L100
[staticcheck] reported by reviewdog 🐶
variadic argument "kvPairs" is expected to have even number of elements, but has 3 elements
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"variadic argument \"kvPairs\" is expected to have even number of elements, but has 3 elements","code":{"value":"SA5012","url":"https://staticcheck.io/docs/checks#SA5012"},"location":{"path":"/home/runner/work/goutil/goutil/envutil/envutil_test.go","range":{"start":{"line":100,"column":11}}},"severity":"ERROR"}
|
Test on go 1.20 and ubuntu-latest:
reflects/reflects.go#L15
[staticcheck] reported by reviewdog 🐶
var fmtStringerType is unused
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"var fmtStringerType is unused","code":{"value":"U1000","url":"https://staticcheck.io/docs/checks#U1000"},"location":{"path":"/home/runner/work/goutil/goutil/reflects/reflects.go","range":{"start":{"line":15,"column":2}}},"severity":"ERROR"}
|
Test on go 1.20 and ubuntu-latest:
reflects/util.go#L13
[staticcheck] reported by reviewdog 🐶
func loopIndirect is unused
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"func loopIndirect is unused","code":{"value":"U1000","url":"https://staticcheck.io/docs/checks#U1000"},"location":{"path":"/home/runner/work/goutil/goutil/reflects/util.go","range":{"start":{"line":13,"column":6}}},"severity":"ERROR"}
|
Test on go 1.20 and ubuntu-latest
Process completed with exit code 1.
|
Test on go 1.20 and windows-latest
The job was canceled because "ubuntu-latest_1_20" failed.
|
Test on go 1.21 and windows-latest
The job was canceled because "ubuntu-latest_1_20" failed.
|
Test on go 1.21 and windows-latest
The operation was canceled.
|
Test on go 1.20 and ubuntu-latest:
encodes/hashutil/hashutil.go#L1
should have a package comment
|
Test on go 1.20 and ubuntu-latest:
encodes/encodes.go#L1
should have a package comment
|
Test on go 1.20 and ubuntu-latest:
fmtutil/format.go#L68
if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary)
|
Test on go 1.20 and ubuntu-latest:
func.go#L49
comment on exported function SafeRunWithError should be of the form "SafeRunWithError ..."
|
Test on go 1.20 and ubuntu-latest:
sysutil/user_nonwin.go#L21
func ChangeUserUidGid should be ChangeUserUIDGid
|
Test on go 1.20 and ubuntu-latest:
dump/dumper_test.go#L194
parameter 't' seems to be unused, consider removing or renaming it as _
|
Test on go 1.20 and ubuntu-latest:
dump/dumper_test.go#L215
parameter 't' seems to be unused, consider removing or renaming it as _
|
Test on go 1.20 and ubuntu-latest:
dump/dumper_test.go#L235
parameter 't' seems to be unused, consider removing or renaming it as _
|
Test on go 1.20 and ubuntu-latest:
dump/dumper_test.go#L291
parameter 't' seems to be unused, consider removing or renaming it as _
|
Test on go 1.20 and ubuntu-latest:
testutil/httpmock.go#L31
func NewHttpRequest should be NewHTTPRequest
|