auth0 / auth0-flutter

Auth0 SDK for Flutter
https://pub.dev/documentation/auth0_flutter/latest/
Apache License 2.0
57 stars 37 forks source link

af: Gate native Swift/Objective-C code per platform [SDK-4700] 1/3 #346

Closed Widcket closed 8 months ago

Widcket commented 8 months ago

📋 Changes

This PR gates all the iOS-specific code and macOS-specific code in auth0_flutter behind compile-time conditionals, so it compiles correctly on both platforms.

Also the Podspec of auth0_flutter was updated to use the FlutterMacOS dependency on macOS, instead of Flutter, which is iOS-only, and the minimum deployment target of the sample app was updated to match the version specified in the Podspec.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (feature/macos-support@c5ee3e7). Click here to learn what that means.

:exclamation: Current head bb1bfbf differs from pull request most recent head 9303d01. Consider uploading reports for the commit 9303d01 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## feature/macos-support #346 +/- ## ======================================================== Coverage ? 96.36% ======================================================== Files ? 98 Lines ? 1596 Branches ? 328 ======================================================== Hits ? 1538 Misses ? 50 Partials ? 8 ``` | [Flag](https://app.codecov.io/gh/auth0/auth0-flutter/pull/346/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | Coverage Δ | | |---|---|---| | [auth0_flutter](https://app.codecov.io/gh/auth0/auth0-flutter/pull/346/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `100.00% <0.00%> (?)` | | | [auth0_flutter_android](https://app.codecov.io/gh/auth0/auth0-flutter/pull/346/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `97.35% <0.00%> (?)` | | | [auth0_flutter_ios](https://app.codecov.io/gh/auth0/auth0-flutter/pull/346/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `99.85% <0.00%> (?)` | | | [auth0_flutter_platform_interface](https://app.codecov.io/gh/auth0/auth0-flutter/pull/346/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0) | `87.03% <0.00%> (?)` | | 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=auth0#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.

Widcket commented 8 months ago

The flutter analyze CI check (CI / Analyze auth0_flutter_platform_interface Flutter package) will pass once https://github.com/auth0/auth0-flutter/pull/347 gets merged.