search

pusher / push-notifications-swift

Swift SDK for the Pusher Beams product:
https://www.pusher.com/beams
MIT License
34 stars 24 forks source link

Resolve some SwiftLint warnings #169

Closed danielrbrowne closed 3 years ago

danielrbrowne commented 3 years ago

As per title.

codecov-io commented 3 years ago

Codecov Report

Merging #169 (ba0dbcb) into master (e752668) will decrease coverage by 0.41%. The diff coverage is 58.13%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #169      +/-   ##
==========================================
- Coverage   90.18%   89.77%   -0.42%     
==========================================
  Files          57       57              
  Lines        2925     2925              
==========================================
- Hits         2638     2626      -12     
- Misses        287      299      +12
Impacted Files Coverage Δ
Sources/PushNotifications.swift 87.80% <ø> (ø)
Sources/PushNotificationsStatic.swift 40.21% <ø> (ø)
Tests/IntegrationTests/DeviceInterestsTest.swift 95.30% <ø> (ø)
...rationTests/MultipleClassInstanceSupportTest.swift 100.00% <ø> (ø)
Sources/API/NetworkService.swift 68.45% <51.72%> (ø)
Sources/Internal/ServerSyncProcessHandler.swift 80.23% <61.90%> (-1.98%) :arrow_down:
Sources/API/PushNotificationsAPIError.swift 83.33% <100.00%> (+8.33%) :arrow_up:
Sources/API/RetryStrategy.swift 100.00% <100.00%> (ø)
Sources/ServerSyncJob.swift 60.82% <0.00%> (-8.25%) :arrow_down:
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 82d45b5...ba0dbcb. Read the comment docs.