Skip to content

🐛 fix: conv - fix some to unit convert tests error #687

🐛 fix: conv - fix some to unit convert tests error

🐛 fix: conv - fix some to unit convert tests error #687

Triggered via push December 8, 2023 06:13
Status Success
Total duration 2m 19s
Artifacts

go.yml

on: push
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 10 warnings
[staticcheck] envutil/envutil_test.go#L102 <SA5012>(https://staticcheck.io/docs/checks#SA5012): envutil/envutil_test.go#L102
variadic argument "kvPairs" is expected to have even number of elements, but has 3 elements
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: 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: dump/dumper_test.go#L361
parameter 't' seems to be unused, consider removing or renaming it as _
Test on go 1.20 and ubuntu-latest: dump/dumper_test.go#L365
parameter 't' seems to be unused, consider removing or renaming it as _
Test on go 1.20 and ubuntu-latest: strutil/convbase.go#L15
exported const Base10Chars should have comment (or a comment on this block) or be unexported
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