Skip to content
Test and Lint

Remove the old “newSessionBegan” signal #241

Sign in to view logs

Remove the old “newSessionBegan” signal

Remove the old “newSessionBegan” signal #241

Triggered via pull request September 16, 2024 08:04
@winsmithwinsmith
opened #183
feature/remove-legacy-session-signal
Status Failure
Total duration 6m 11s
Artifacts –

ci.yaml

on: pull_request
Lint Code
25s
Lint Code
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

3 errors
Lint Code: Sources/TelemetryClient/SignalManager.swift#L140
Prefer failable `String(data:encoding:)` initializer when converting `Data` to `String` (optional_data_string_conversion)
Lint Code: Sources/TelemetryClient/SignalManager.swift#L199
Prefer failable `String(data:encoding:)` initializer when converting `Data` to `String` (optional_data_string_conversion)
Lint Code: Sources/TelemetryClient/Signal.swift#L215
Prefer failable `String(data:encoding:)` initializer when converting `Data` to `String` (optional_data_string_conversion)