Skip to content
reviewdog

feat: otel tracing #6141

Sign in to view logs

feat: otel tracing

feat: otel tracing #6141

Triggered via pull request August 20, 2024 14:05
@plyr4plyr4
synchronize #899
feat/otel-tracing
Status Failure
Total duration 2m 42s
Artifacts –

reviewdog.yml

on: pull_request
diff-review
2m 17s
diff-review
full-review
2m 28s
full-review
Fit to window
Zoom out
Zoom in

Annotations

20 errors
diff-review: mock/server/authentication.go#L81
string `error` has 4 occurrences, make it a constant (goconst)
diff-review: database/executable/create.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/executable/pop.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/executable/table.go#L38
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: tracing/sampler/sampler.go#L43
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
diff-review: tracing/sampler/sampler.go#L34
unused-parameter: parameter 'parameters' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: tracing/sampler/sampler.go#L13
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
diff-review: tracing/sampler/sampler.go#L14
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
diff-review: tracing/sampler/sampler.go#L15
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
diff-review: secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
full-review: mock/server/authentication.go#L81
string `error` has 4 occurrences, make it a constant (goconst)
full-review: database/executable/clean.go#L31
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/executable/create.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/executable/pop.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: tracing/sampler/sampler.go#L43
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
full-review: tracing/sampler/sampler.go#L13
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
full-review: tracing/sampler/sampler.go#L14
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
full-review: tracing/sampler/sampler.go#L15
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
full-review: tracing/sampler/sampler.go#L34
unused-parameter: parameter 'parameters' seems to be unused, consider removing or renaming it as _ (revive)
full-review: secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)