search

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

Clean up the project by fixing SwiftLint warnings #147

Closed AvdLee closed 1 year ago

AvdLee commented 1 year ago

It turned out we had quite some warnings to solve based on SwiftLint feedback.

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

Code Coverage Report

Name Coverage
Diagnostics 69.14% ⚠️

SwiftLint found issues

Severity File Reason
Warning AppDelegate.swift:8 The disabled 'line_length' rule should be re-enabled before the end of the file (blanket_disable_command)
Warning DiagnosticsReporter.swift:8 The disabled 'line_length' rule should be re-enabled before the end of the file (blanket_disable_command)

Generated by :no_entry_sign: Danger Swift against cfdb1edc589f18e481a6fdde2530afffdc3c1f45

wetransferplatform commented 1 year ago

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

Generated by GitBuddy