Serverless verify

Stop using /common/logging from kyma monorepo #394

Sign in to view logs

Triggered via pull request February 22, 2024 14:44

kwiatekus

opened #742

kwiatekus:stop-usin-common/logging

Status Failure

Total duration 6m 38s

Artifacts –

serverless-verify.yaml

on: pull_request

contract-integration-test

git-auth-integration-test

Annotations

10 errors and 1 warning

lint: components/serverless/internal/logging/tracing/helper.go#L6

field `traceID` is unused (unused)

lint: components/serverless/internal/logging/tracing/helper.go#L7

field `spanID` is unused (unused)

lint: components/serverless/internal/logging/logger/logger_test.go#L129

S1030: should use buf.String() instead of string(buf.Bytes()) (gosimple)

lint: components/serverless/internal/logging/logger/format.go#L26

S1028: should use fmt.Errorf(...) instead of errors.New(fmt.Sprintf(...)) (gosimple)

lint: components/serverless/internal/logging/logger/level.go#L30

S1028: should use fmt.Errorf(...) instead of errors.New(fmt.Sprintf(...)) (gosimple)

lint: components/serverless/internal/logging/tracing/middleware.go#L32

S1023: redundant `return` statement (gosimple)

lint: components/serverless/internal/logging/logger/logger_test.go#L86

ineffectual assignment to err (ineffassign)

lint: components/serverless/internal/logging/logger/logger_test.go#L125

ineffectual assignment to err (ineffassign)

lint: components/serverless/internal/logging/logger/logger_test.go#L153

ineffectual assignment to err (ineffassign)

lint: components/serverless/internal/logging/tracing/middleware.go#L41

SA1029: should not use built-in type string as key for value; define your own type to avoid collisions (staticcheck)

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.