aws-amplify / amplify-swift

A declarative library for application development using cloud services.
Apache License 2.0
456 stars 198 forks source link

chore(logging): resolve swiftformat errors and warnings #3850

Closed phantumcode closed 1 month ago

phantumcode commented 2 months ago

Issue \

Description

Resolving SwiftFormat errors and warnings for the Logging plugin.

General Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 76.02339% with 41 lines in your changes missing coverage. Please review.

Project coverage is 68.53%. Comparing base (ca6c65a) to head (e665bee).

Files with missing lines Patch % Lines
...Plugin/AWSCloudWatchLoggingSessionController.swift 50.00% 15 Missing :warning:
...atchLoggingPlugin/AWSCloudWatchLoggingPlugin.swift 0.00% 11 Missing :warning:
...tion/DefaultRemoteLoggingConstraintsProvider.swift 0.00% 7 Missing :warning:
...ingPlugin/AWSCloudWatchLoggingCategoryClient.swift 82.60% 4 Missing :warning:
...SCloudWatchLoggingPlugin/Persistence/LogFile.swift 66.66% 2 Missing :warning:
...udWatchLoggingPlugin/Producer/RotatingLogger.swift 75.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## chore/resolve_swiftformat #3850 +/- ## ============================================================= + Coverage 68.51% 68.53% +0.02% ============================================================= Files 1080 1080 Lines 37608 37638 +30 ============================================================= + Hits 25766 25796 +30 Misses 11842 11842 ``` | [Flag](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | Coverage Δ | | |---|---|---| | [API_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.96% <ø> (ø)` | | | [AWSPluginsCore](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.56% <ø> (ø)` | | | [Amplify](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `47.56% <ø> (ø)` | | | [Analytics_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `84.52% <ø> (ø)` | | | [Auth_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `79.67% <ø> (-0.01%)` | :arrow_down: | | [DataStore_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `81.27% <ø> (+0.05%)` | :arrow_up: | | [Geo_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `72.00% <ø> (ø)` | | | [Logging_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `63.77% <76.02%> (+0.81%)` | :arrow_up: | | [Predictions_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `37.32% <ø> (ø)` | | | [PushNotifications_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `86.21% <ø> (ø)` | | | [Storage_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `76.81% <ø> (ø)` | | | [unit_tests](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3850/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.53% <76.02%> (+0.02%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.