build(deps): bump github.com/gookit/goutil from 0.6.15 to 0.6.16 #439
Annotations
12 errors and 30 warnings
Test on go 1.19 and ubuntu-latest:
home/runner/go/pkg/mod/github.com/gorilla/[email protected]/server.go#L176
[staticcheck] reported by reviewdog 🐶
undefined: http.NewResponseController
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"undefined: http.NewResponseController","code":{"value":"compile","url":"https://staticcheck.io/docs/checks#compile"},"location":{"path":"/home/runner/go/pkg/mod/github.com/gorilla/[email protected]/server.go","range":{"start":{"line":176,"column":28},"end":{"line":0,"column":0}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
benchmark_test.go#L81
[staticcheck] reported by reviewdog 🐶
empty branch
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"empty branch","code":{"value":"SA9003","url":"https://staticcheck.io/docs/checks#SA9003"},"location":{"path":"/home/runner/work/rux/rux/benchmark_test.go","range":{"start":{"line":81,"column":3},"end":{"line":81,"column":36}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
benchmark_test.go#L81
[staticcheck] reported by reviewdog 🐶
this value of ok is never used
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"this value of ok is never used","code":{"value":"SA4006","url":"https://staticcheck.io/docs/checks#SA4006"},"location":{"path":"/home/runner/work/rux/rux/benchmark_test.go","range":{"start":{"line":81,"column":6},"end":{"line":81,"column":32}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
benchmark_test.go#L98
[staticcheck] reported by reviewdog 🐶
empty branch
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"empty branch","code":{"value":"SA9003","url":"https://staticcheck.io/docs/checks#SA9003"},"location":{"path":"/home/runner/work/rux/rux/benchmark_test.go","range":{"start":{"line":98,"column":3},"end":{"line":98,"column":69}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
context.go#L188
[staticcheck] reported by reviewdog 🐶
unnecessary assignment to the blank identifier
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"unnecessary assignment to the blank identifier","code":{"value":"S1005","url":"https://staticcheck.io/docs/checks#S1005"},"location":{"path":"/home/runner/work/rux/rux/context.go","range":{"start":{"line":188,"column":5},"end":{"line":188,"column":35}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
dispatch.go#L32
[staticcheck] reported by reviewdog 🐶
should use constant http.StatusMethodNotAllowed instead of numeric literal 405
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"should use constant http.StatusMethodNotAllowed instead of numeric literal 405","code":{"value":"ST1013","url":"https://staticcheck.io/docs/checks#ST1013"},"location":{"path":"/home/runner/work/rux/rux/dispatch.go","range":{"start":{"line":32,"column":44},"end":{"line":32,"column":47}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
pkg/binding/binder.go#L78
[staticcheck] reported by reviewdog 🐶
unnecessary guard around call to delete
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"unnecessary guard around call to delete","code":{"value":"S1033","url":"https://staticcheck.io/docs/checks#S1033"},"location":{"path":"/home/runner/work/rux/rux/pkg/binding/binder.go","range":{"start":{"line":78,"column":3},"end":{"line":78,"column":32}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
pkg/handlers/handlers_test.go#L88
[staticcheck] reported by reviewdog 🐶
should use 'return c.URL().Path == "/test4"' instead of 'if c.URL().Path == "/test4" { return true }; return false'
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"should use 'return c.URL().Path == \"/test4\"' instead of 'if c.URL().Path == \"/test4\" { return true }; return false'","code":{"value":"S1008","url":"https://staticcheck.io/docs/checks#S1008"},"location":{"path":"/home/runner/work/rux/rux/pkg/handlers/handlers_test.go","range":{"start":{"line":88,"column":3},"end":{"line":90,"column":4}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L180
[staticcheck] reported by reviewdog 🐶
func restoreStdout is unused
Raw Output:
{"source":{"name":"staticcheck","url":"https://staticcheck.io"},"message":"func restoreStdout is unused","code":{"value":"U1000","url":"https://staticcheck.io/docs/checks#U1000"},"location":{"path":"/home/runner/work/rux/rux/pkg/handlers/middlewares_test.go","range":{"start":{"line":180,"column":6},"end":{"line":0,"column":0}}},"related_locations":[],"severity":"ERROR"}
|
Test on go 1.19 and ubuntu-latest
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.com/orgs/community/discussions/26680#discussioncomment-3252835
|
Test on go 1.20 and ubuntu-latest
The job was canceled because "ubuntu-latest_1_19" failed.
|
Test on go 1.18 and ubuntu-latest
The job was canceled because "ubuntu-latest_1_19" failed.
|
Test on go 1.19 and ubuntu-latest:
server/echo.go#L1
should have a package comment
|
Test on go 1.19 and ubuntu-latest:
pkg/pprof/pprof.go#L1
should have a package comment
|
Test on go 1.19 and ubuntu-latest:
pkg/render/json.go#L1
should have a package comment
|
Test on go 1.19 and ubuntu-latest:
pkg/render/template.go#L15
parameter 'obj' seems to be unused, consider removing or renaming it as _
|
Test on go 1.19 and ubuntu-latest:
pkg/binding/form.go#L39
func DecodeUrlValues should be DecodeURLValues
|
Test on go 1.19 and ubuntu-latest:
pkg/binding/query.go#L8
comment on exported var QueryTagName should be of the form "QueryTagName ..."
|
Test on go 1.19 and ubuntu-latest:
pkg/handlers/console_logger.go#L1
should have a package comment
|
Test on go 1.19 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L41
parameter 'c' seems to be unused, consider removing or renaming it as _
|
Test on go 1.19 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L36
parameter 'c' seems to be unused, consider removing or renaming it as _
|
Test on go 1.19 and ubuntu-latest:
context.go#L455
var isHttp should be isHTTP
|
Test on go 1.20 and ubuntu-latest:
pkg/handlers/console_logger.go#L1
should have a package comment
|
Test on go 1.20 and ubuntu-latest:
pkg/handlers/console_logger.go#L13
exported var IgnorePaths should have comment or be unexported
|
Test on go 1.20 and ubuntu-latest:
server/echo.go#L1
should have a package comment
|
Test on go 1.20 and ubuntu-latest:
pkg/pprof/pprof.go#L1
should have a package comment
|
Test on go 1.20 and ubuntu-latest:
pkg/binding/query.go#L8
comment on exported var QueryTagName should be of the form "QueryTagName ..."
|
Test on go 1.20 and ubuntu-latest:
pkg/render/json.go#L1
should have a package comment
|
Test on go 1.20 and ubuntu-latest:
pkg/render/template.go#L15
parameter 'obj' seems to be unused, consider removing or renaming it as _
|
Test on go 1.20 and ubuntu-latest:
pkg/binding/form.go#L39
func DecodeUrlValues should be DecodeURLValues
|
Test on go 1.20 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L41
parameter 'c' seems to be unused, consider removing or renaming it as _
|
Test on go 1.20 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L36
parameter 'c' seems to be unused, consider removing or renaming it as _
|
Test on go 1.18 and ubuntu-latest:
pkg/render/json.go#L1
should have a package comment
|
Test on go 1.18 and ubuntu-latest:
server/echo.go#L1
should have a package comment
|
Test on go 1.18 and ubuntu-latest:
pkg/binding/query.go#L8
comment on exported var QueryTagName should be of the form "QueryTagName ..."
|
Test on go 1.18 and ubuntu-latest:
pkg/render/template.go#L15
parameter 'obj' seems to be unused, consider removing or renaming it as _
|
Test on go 1.18 and ubuntu-latest:
pkg/binding/form.go#L39
func DecodeUrlValues should be DecodeURLValues
|
Test on go 1.18 and ubuntu-latest:
pkg/pprof/pprof.go#L1
should have a package comment
|
Test on go 1.18 and ubuntu-latest:
pkg/handlers/console_logger.go#L1
should have a package comment
|
Test on go 1.18 and ubuntu-latest:
pkg/handlers/console_logger.go#L13
exported var IgnorePaths should have comment or be unexported
|
Test on go 1.18 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L41
parameter 'c' seems to be unused, consider removing or renaming it as _
|
Test on go 1.18 and ubuntu-latest:
pkg/handlers/middlewares_test.go#L36
parameter 'c' seems to be unused, consider removing or renaming it as _
|