fix(slog): use correct slog attributes for errors #94
pkkummermoAnnotations
4 errors and 11 warnings
|
lint:
server.go#L329
printf: non-constant format string in call to fmt.Errorf (govet)
|
|
lint:
server.go#L328
SA1006: printf-style function with dynamic format string and no further arguments should use print-style function instead (staticcheck)
|
|
lint:
call.go#L89
SA1032: arguments have the wrong order (staticcheck)
|
|
lint
issues found
|
|
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint:
ws.go#L31
unused-parameter: parameter 'wsCall' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
ws.go#L32
unused-parameter: parameter 'wsMessage' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
ws.go#L33
unused-parameter: parameter 'closeCode' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L14
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L30
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L52
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L68
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L84
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L101
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
plugins/cors/cors_test.go#L118
unused-parameter: parameter 'app' seems to be unused, consider removing or renaming it as _ (revive)
|