-
Notifications
You must be signed in to change notification settings - Fork 29.2k
Roll pub packages #115866
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 #115866
Conversation
auto label is removed for flutter/flutter, pr: 115866, due to - Please get at least one approved review if you are already a member or two member reviews if you are not a member before re-applying this label. Reviewers: If you left a comment approving, please use the "approve" review action instead. |
auto label is removed for flutter/flutter, pr: 115866, due to Validations Fail. |
Build tests failure is https://ci.chromium.org/ui/p/flutter/builders/try/Linux%20build_tests_2_3/1012/overview:
|
@stuartmorgan do you have any idea why your change in flutter/plugins#6744 is causing gradle version solving to fail? Do we have to manually update something in the test apps' android directories? |
No idea, but this is the dependency that I removed. It's mentioning the "lock state", and I see that there are gradle lock files referencing it. I don't know anything about Gradle lock files, but it seems like they are probably the issue here. |
cc @reidbaker is the fix here to just one time re-generate https://github.com/flutter/flutter/blob/master/dev/integration_tests/hybrid_android_views/android/project-path_provider.lockfile ? I was thinking maybe we could just not check it in to version control, but the comment at the top says its intended to be version controlled. |
I'm going to file an issue as it looks like the number of lockfiles we have in the repo is non-trivial |
nvm, turns out there's already a script in the repo for this: #92800 |
Obsolete, manually pinned and rolled in #116377 |
This PR was generated by
flutter update-packages --force-upgrade
.