-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Roll pub packages #126966
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
Roll pub packages #126966
Conversation
|
auto label is removed for flutter/flutter, pr: 126966, due to This PR has not met approval requirements for merging. You are not a member of flutter-hackers and need 2 more review(s) in order to merge this PR.
|
|
@natebosch there are 1878 analysis failures across the repo (none in flutter_tools though): https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8780898069202916497/+/u/Analyze/analyze/test_stdout Was test v1.24.3 expected to be this breaking? |
|
No, I cannot find where the matchers are exported from |
|
The APIs which appear to be missing should be getting exported from
|
|
I see an error This PR does bump |
packages/flutter_test/pubspec.yaml
Outdated
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.
This looks off - matcher should remain a dependency of flutter_test.
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.
ahh, good catch. I guess flutter_test now needs to directly depend on it, rather than transitively via test_api.
6341cb3 to
e8210ef
Compare
|
Ok, now the problem seems to be a conflict in customer_testing with apps that depend on mockito: Given |
Published in version |
nice, thanks! |
This PR was generated by
flutter update-packages --force-upgrade.