Skip to content

Conversation

@stuartmorgan-g
Copy link
Collaborator

Updates to use local_auth_darwin instead of the discontinued local_auth_ios.

Pre-launch Checklist

@stuartmorgan-g stuartmorgan-g marked this pull request as draft February 12, 2024 19:25
@stuartmorgan-g stuartmorgan-g changed the title Switch to local_auth_darwin [local_auth] Switch iOS endorsement to local_auth_darwin Feb 12, 2024
@stuartmorgan-g

This comment was marked as resolved.

@stuartmorgan-g

This comment was marked as resolved.

@stuartmorgan-g

This comment was marked as resolved.

@stuartmorgan-g stuartmorgan-g marked this pull request as ready for review February 16, 2024 17:38
@stuartmorgan-g
Copy link
Collaborator Author

I was hoping to have macOS support land in local_auth_darwin first, but that PR isn't ready yet, and I don't want us to stay in the limbo of depending on a package we no longer have the ability to push fixes to any longer.

* Clients directly importing `local_auth_ios` for auth strings should switch
dependencies and imports to `local_auth_darwin`. No other change is necessary.
* Updates support matrix in README to indicate that iOS 11 is no longer supported.
* Clients on versions of Flutter that still support iOS 11 can continue to use this
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed this because it no longer makes sense; with 3.16.6 as the minimum, nobody supporting iOS 11 can use this version of local_auth anyway.

Copy link
Contributor

@LouiseHsu LouiseHsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@stuartmorgan-g stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 20, 2024
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Feb 20, 2024
@auto-submit
Copy link
Contributor

auto-submit bot commented Feb 20, 2024

auto label is removed for flutter/packages/6107, due to - The status or check suite Linux_android android_platform_tests_api_33_shard_2 master has failed. Please fix the issues identified (or deflake) before re-applying this label.

@stuartmorgan-g stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 20, 2024
@auto-submit
Copy link
Contributor

auto-submit bot commented Feb 20, 2024

auto label is removed for flutter/packages/6107, due to - The status or check suite Linux_android android_platform_tests_api_33_shard_2 master has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Feb 20, 2024
@stuartmorgan-g stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 20, 2024
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Feb 20, 2024
@auto-submit
Copy link
Contributor

auto-submit bot commented Feb 20, 2024

auto label is removed for flutter/packages/6107, due to - The status or check suite Linux_android android_platform_tests_shard_6 master has failed. Please fix the issues identified (or deflake) before re-applying this label.

@stuartmorgan-g stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 21, 2024
@auto-submit auto-submit bot merged commit cd5d7d8 into flutter:main Feb 21, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 22, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Feb 22, 2024
flutter/packages@48048f6...078c2a3

2024-02-22 [email protected] Manual roll Flutter from 5129806 to efee280 (47 revisions) (flutter/packages#6180)
2024-02-22 [email protected] Manual roll Flutter (stable) from bae5e49 to abb292a (1 revision) (flutter/packages#6179)
2024-02-21 [email protected] [in_app_purchase] Convert refreshReceipt(), startObservingPaymentQueue(), stopObservingPaymentQueue(), registerPaymentQueueDelegate(), removePaymentQueueDelegate(), showPriceConsentIfNeeded() to Pigeon (flutter/packages#6165)
2024-02-21 [email protected] [camerax] Modifies initialized camera info to reflect default AF/AE modes and the ability to set focus points for each (flutter/packages#6109)
2024-02-21 [email protected] [camerax] Implements `setFocusPoint`, `setExposurePoint`, `setExposureOffset` (flutter/packages#6059)
2024-02-21 [email protected] [various] Commit Windows build migrations (flutter/packages#6175)
2024-02-21 [email protected] [flutter_markdown] Support wasm (flutter/packages#6168)
2024-02-21 [email protected] [ci] Run Windows Arm64 build tests post-submit (flutter/packages#6166)
2024-02-21 [email protected] [url_launcher] Remove `renderView` usage (flutter/packages#6137)
2024-02-21 [email protected] [camera_android_camerax] Fix typo in readme (flutter/packages#6143)
2024-02-21 [email protected] [local_auth] Switch iOS endorsement to `local_auth_darwin` (flutter/packages#6107)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App p: local_auth

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants