WeTransfer / Diagnostics

Allow users to easily share Diagnostics with your support team to improve the flow of fixing bugs.
MIT License
933 stars 53 forks source link

Add Sendable conformance #146

Closed AvdLee closed 1 year ago

AvdLee commented 1 year ago

Adds Sendable conformance to our types that aren't mutable.

wetransferplatform commented 1 year ago
Messages
:book: DiagnosticsTests: Executed 33 tests (0 failed, 0 retried, 0 skipped) in 10.347 seconds
:book: Slowest test: DiagnosticsReporterTests/testReportingChapters() (2.028s)
:book: Slowest test: DiagnosticsReporterTests/testWithoutProvidingSmartInsightsProvider() (2.007s)
:book: Slowest test: DiagnosticsReporterTests/testWithSmartInsightsProviderReturningNoExtraInsights() (2.007s)
:book: View more details on Bitrise

Code Coverage Report

Name Coverage
Diagnostics 69.24% ⚠️

SwiftLint found issues

Severity File Reason
Warning HTMLGenerating.swift:72 Use shorthand syntax for optional binding (shorthand_optional_binding)
Warning DiagnosticsReporterTests.swift:93 Variable declarations should be separated from other statements by a blank line (let_var_whitespace)
Warning DiagnosticsReporterTests.swift:58 Line should be 140 characters or less; currently it has 153 characters (line_length)
Error DiagnosticsReporterTests.swift:65 Line should be 160 characters or less; currently it has 194 characters (line_length)
Error DiagnosticsReporterTests.swift:73 Line should be 160 characters or less; currently it has 206 characters (line_length)
Warning DiagnosticsReporterTests.swift:108 Use shorthand syntax for optional binding (shorthand_optional_binding)
Warning DiagnosticsReporterTests.swift:10 Imports should be sorted (sorted_imports)

Generated by :no_entry_sign: Danger Swift against cd1c684c074b3ee5acd294913180a7a903a660b5

wetransferplatform commented 1 year ago

Congratulations! :tada: This was released as part of Release 4.4.1 :rocket:

Generated by GitBuddy