-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat!: Migrate firebase_ml_vision to sound null safety #5474
feat!: Migrate firebase_ml_vision to sound null safety #5474
Conversation
Update fork
@AlexHartford would you mind resolving the conflicts, sorry. I took a look at doing it myself but wasn't too familiar with the changes done so didn't want to break your PR 🙃 |
Handled the merge conflicts. Let me know if there's anything else you need! 😁 |
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.
Sorry for the delay, LGTM!
Any plans to complete the merge soon? |
This has now been published to pub.dev - thanks everyone. |
Description
I only have an Android emulator available to me for the next couple of weeks so it'd be good if someone could test the example project on iOS.
Related Issues
#4182
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
).This will ensure a smooth and quick review process. Updating the
pubspec.yaml
and changelogs is not required.///
).flutter analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?