👔 up: maputil - fix a method name error, add tests for MergeM… #685
inhereAnnotations
10 warnings
|
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:
encodes/hashutil/hashutil.go#L1
should have a package comment
|
|
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:
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:
strutil/check.go#L293
if block ends with a return statement, so drop this else and outdent its block
|
|
Test on go 1.20 and ubuntu-latest:
testutil/httpmock.go#L31
func NewHttpRequest should be NewHTTPRequest
|
|
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:
func.go#L49
comment on exported function SafeRunWithError should be of the form "SafeRunWithError ..."
|