aws-amplify / amplify-swift

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

chore(core): resolve swiftformat errors and warnings #3847

Open phantumcode opened 3 weeks ago

phantumcode commented 3 weeks ago

Issue \

Description

Resolving SwiftFormat errors and warnings for Amplify Core.

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 weeks ago

Codecov Report

Attention: Patch coverage is 77.09611% with 112 lines in your changes missing coverage. Please review.

Project coverage is 67.75%. Comparing base (ca6c65a) to head (c418da3). Report is 2 commits behind head on chore/resolve_swiftformat.

Files with missing lines Patch % Lines
...re/Model/GraphQLRequest/GraphQLRequest+Model.swift 78.22% 27 Missing :warning:
...gins/Core/AWSPluginsCore/Auth/AWSAuthService.swift 0.00% 18 Missing :warning:
...aphQLRequest/GraphQLRequest+AnyModelWithSync.swift 89.81% 11 Missing :warning:
...WSPluginsCore/Sync/MutationSync/MutationSync.swift 46.66% 8 Missing :warning:
.../Configuration/AWSAuthorizationConfiguration.swift 0.00% 7 Missing :warning:
...ore/AWSPluginsCore/WebSocket/WebSocketClient.swift 80.64% 6 Missing :warning:
.../Core/AWSPluginsCore/Model/AnyModel/AnyModel.swift 0.00% 5 Missing :warning:
...insCore/Model/Support/QueryPredicate+GraphQL.swift 68.75% 5 Missing :warning:
...SPluginsCore/Auth/AuthAWSCredentialsProvider.swift 0.00% 4 Missing :warning:
...PluginsCore/Model/Decorator/ModelIdDecorator.swift 60.00% 4 Missing :warning:
... and 15 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## chore/resolve_swiftformat #3847 +/- ## ============================================================= - Coverage 68.51% 67.75% -0.76% ============================================================= Files 1080 1006 -74 Lines 37608 34296 -3312 ============================================================= - Hits 25766 23237 -2529 + Misses 11842 11059 -783 ``` | [Flag](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/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/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.99% <ø> (+0.03%)` | :arrow_up: | | [AWSPluginsCore](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.88% <77.09%> (+0.32%)` | :arrow_up: | | [Amplify](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/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/3847/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/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `79.78% <ø> (+0.10%)` | :arrow_up: | | [DataStore_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `81.29% <ø> (+0.07%)` | :arrow_up: | | [Geo_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/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/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `62.95% <ø> (ø)` | | | [Predictions_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/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/3847/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/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `?` | | | [unit_tests](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3847/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `67.75% <77.09%> (-0.76%)` | :arrow_down: | 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.