Build and Test SDK

chore: add fdv2 store & update sources to use fdv2 protocol definitions #277

Sign in to view logs

Triggered via pull request October 8, 2024 15:47

cwaldren-ld

synchronize #192

cw/sdk-557/fdv2-store

Status Failure

Total duration 4m 20s

Artifacts 6

ci.yml

on: pull_request

go-versions / go-versions

Matrix: test-linux

Matrix: test-windows

Annotations

33 errors

Linux, Go 1.18 / Unit Tests and Coverage: internal/fdv2proto/events.go#L19

commentFormatting: put a space between `//` and comment text (gocritic)

Linux, Go 1.18 / Unit Tests and Coverage: internal/fdv2proto/event_to_storable_item.go#L8

exported: exported function ToStorableItems should have comment or be unexported (revive)

Linux, Go 1.18 / Unit Tests and Coverage: internal/fdv2proto/payloads.go#L3

exported: exported type Payload should have comment or be unexported (revive)

Linux, Go 1.18 / Unit Tests and Coverage: internal/fdv2proto/payloads.go#L15

exported: exported type PollingPayload should have comment or be unexported (revive)

Linux, Go 1.18 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L35

const `putEventName` is unused (unused)

Linux, Go 1.18 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L36

const `deleteEventName` is unused (unused)

Linux, Go 1.18 / Unit Tests and Coverage: internal/datasourcev2/polling_http_request.go#L175

unnecessary trailing newline (whitespace)

Linux, Go 1.18 / Unit Tests and Coverage: internal/fdv2proto/events.go#L79

directive `//nolint:revive // Error method.` is unused for linter "revive" (nolintlint)

Linux, Go 1.18 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L139

directive `//nolint:gocyclo,godox // this function is a stepping stone. It will get better over time.` is unused for linter "gocyclo" (nolintlint)

Linux, Go 1.18 / Unit Tests and Coverage

Process completed with exit code 2.

Linux, Go 1.22 / Unit Tests and Coverage: internal/fdv2proto/events.go#L19

commentFormatting: put a space between `//` and comment text (gocritic)

Linux, Go 1.22 / Unit Tests and Coverage: internal/fdv2proto/event_to_storable_item.go#L8

exported: exported function ToStorableItems should have comment or be unexported (revive)

Linux, Go 1.22 / Unit Tests and Coverage: internal/fdv2proto/payloads.go#L3

exported: exported type Payload should have comment or be unexported (revive)

Linux, Go 1.22 / Unit Tests and Coverage: internal/fdv2proto/payloads.go#L15

exported: exported type PollingPayload should have comment or be unexported (revive)

Linux, Go 1.22 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L35

const `putEventName` is unused (unused)

Linux, Go 1.22 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L36

const `deleteEventName` is unused (unused)

Linux, Go 1.22 / Unit Tests and Coverage: internal/datasourcev2/polling_http_request.go#L175

unnecessary trailing newline (whitespace)

Linux, Go 1.22 / Unit Tests and Coverage: internal/fdv2proto/events.go#L79

directive `//nolint:revive // Error method.` is unused for linter "revive" (nolintlint)

Linux, Go 1.22 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L139

directive `//nolint:gocyclo,godox // this function is a stepping stone. It will get better over time.` is unused for linter "gocyclo" (nolintlint)

Linux, Go 1.22 / Unit Tests and Coverage

Process completed with exit code 2.

Linux, Go 1.23 / Unit Tests and Coverage: internal/fdv2proto/events.go#L19

commentFormatting: put a space between `//` and comment text (gocritic)

Linux, Go 1.23 / Unit Tests and Coverage: internal/fdv2proto/event_to_storable_item.go#L8

exported: exported function ToStorableItems should have comment or be unexported (revive)

Linux, Go 1.23 / Unit Tests and Coverage: internal/fdv2proto/payloads.go#L3

exported: exported type Payload should have comment or be unexported (revive)

Linux, Go 1.23 / Unit Tests and Coverage: internal/fdv2proto/payloads.go#L15

exported: exported type PollingPayload should have comment or be unexported (revive)

Linux, Go 1.23 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L35

const `putEventName` is unused (unused)

Linux, Go 1.23 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L36

const `deleteEventName` is unused (unused)

Linux, Go 1.23 / Unit Tests and Coverage: internal/datasourcev2/polling_http_request.go#L175

unnecessary trailing newline (whitespace)

Linux, Go 1.23 / Unit Tests and Coverage: internal/fdv2proto/events.go#L79

directive `//nolint:revive // Error method.` is unused for linter "revive" (nolintlint)

Linux, Go 1.23 / Unit Tests and Coverage: internal/datasourcev2/streaming_data_source.go#L139

directive `//nolint:gocyclo,godox // this function is a stepping stone. It will get better over time.` is unused for linter "gocyclo" (nolintlint)

Linux, Go 1.23 / Unit Tests and Coverage

Process completed with exit code 2.

Windows, Go 1.18

Process completed with exit code 1.

Windows, Go 1.23

Process completed with exit code 1.

Windows, Go 1.22

Process completed with exit code 1.

Artifacts

Produced during runtime

Name	Size
Benchmarks-go1.18.10	3.14 KB
Benchmarks-go1.22.7	3.15 KB
Benchmarks-go1.23.2	3.16 KB
Contract-test-service-logs-go1.18.10	177 KB
Contract-test-service-logs-go1.22.7	177 KB
Contract-test-service-logs-go1.23.2	177 KB