refactor: create re-usable CompileAndPublish function
#5523
ecrupperreviewdog.yml
on: pull_request
diff-review
32s
full-review
41s
Annotations
20 errors
|
diff-review:
database/executable/create.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
database/executable/pop.go#L15
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:
compiler/registry/github/github.go#L41
string `/api/v3/` has 5 occurrences, make it a constant (goconst)
|
|
diff-review:
database/resource.go#L29
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
diff-review:
database/resource.go#L40
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
diff-review:
database/resource.go#L53
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
diff-review:
scm/github/authentication.go#L58
unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
scm/github/access.go#L34
Function `newClientToken` should pass the context parameter (contextcheck)
|
|
diff-review:
scm/github/access.go#L103
Function `newClientToken` should pass the context parameter (contextcheck)
|
|
full-review:
database/repo/create.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
database/repo/table.go#L84
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
database/repo/count.go#L12
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
database/resource.go#L29
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
full-review:
database/resource.go#L40
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
full-review:
database/resource.go#L53
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
full-review:
queue/redis/redis_test.go#L95
`_steps` is unused (deadcode)
|
|
full-review:
queue/redis/route_test.go#L62
G601: Implicit memory aliasing in for loop. (gosec)
|
|
full-review:
router/middleware/logger.go#L42
unused-parameter: parameter 'timeFormat' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
router/middleware/header.go#L37
expressions should not be cuddled with blocks (wsl)
|