-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[local_auth] Update to pigeon 11 and remove enum wrappers #4809
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! It's great to see all that manual boilerplate go away.
@@ -79,7 +79,7 @@ void main() { | |||
]); | |||
}); | |||
|
|||
test('handles emtpy', () async { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we sure we don't need test coverage of "emtpy" handling?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's a risk we're going to have to take unfortunately. If we see a flare up of emtpy in the future we can revisit this at that time.
flutter/packages@64af59e...e7d812c 2023-08-30 [email protected] [path_provider] Fix93198: Added getDownloadsDirectory() for Android (flutter/packages#4708) 2023-08-30 [email protected] [in_app_purchase] Minor lint cleanup (flutter/packages#4818) 2023-08-30 [email protected] [local_auth] Update to pigeon 11 and remove enum wrappers (flutter/packages#4809) 2023-08-30 [email protected] [image_picker] Fix link in README file. (flutter/packages#4775) 2023-08-30 [email protected] [web] Use new APIs from `dart:ui_web` (flutter/packages#4168) 2023-08-30 [email protected] Roll Flutter from 6c95737 to 1fe2495 (24 revisions) (flutter/packages#4817) 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
updates pigeon version to 11 to remove need for enum wrappers - removes enum wrappers.
Some work toward flutter/flutter#133455