Skip to content

Conversation

@gmackall
Copy link
Member

@gmackall gmackall commented Jun 14, 2023

Unpins flutter_plugin_android_lifecycle where it is pinned. I then

  1. ran flutter update-packages --force-upgrade (but only committed the changes within dev/integration_tests/gradle_deprecated_settings/, which is where it had been pinned)
  2. followed by ./gradlew :generateLockfiles from dev/integration_tests/gradle_deprecated_settings/android/ (the lockfile was what was causing the CI dependency resolution failure, so this second step is the fix for that).

See the reason it was pinned: #121847 (comment) followed by the PR that pinned it: #122043

Fixes #122039

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Jun 14, 2023
@gmackall gmackall marked this pull request as draft June 14, 2023 19:47
@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Jun 14, 2023
@gmackall gmackall changed the title running 'flutter update-packages --force-upgrade' and then './gradlew… Unpin flutter_plugin_android_lifecycle Jun 14, 2023
@gmackall
Copy link
Member Author

I haven't changed these files before so I tried to be explicit in what was done, let me know if anything about the process sounds off 🙂

@gmackall gmackall marked this pull request as ready for review June 14, 2023 20:44
Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM, thanks so much for this!

@gmackall gmackall added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 14, 2023
@auto-submit auto-submit bot merged commit 944d6c8 into flutter:master Jun 14, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 15, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 15, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jun 15, 2023
flutter/flutter@95be76a...b0188cd

2023-06-15 [email protected] [web] Pass creation params to the platform view factory (flutter/flutter#128146)
2023-06-15 [email protected] [flutter_tools] cache flutter sdk version to disk (flutter/flutter#124558)
2023-06-14 [email protected] Fix inconsistently suffixed macOS flavored bundle directory (flutter/flutter#127997)
2023-06-14 [email protected] Update golden tests for material (flutter/flutter#128839)
2023-06-14 [email protected] Update getChildrenSummaryTree to handle Diagnosticable as input. (flutter/flutter#128833)
2023-06-14 [email protected] Improve the error message for non-normalized constraints (flutter/flutter#127906)
2023-06-14 [email protected] ContextAction.isEnabled needs a context (flutter/flutter#127721)
2023-06-14 [email protected] Remove temporary default case for PointerSignalKind (flutter/flutter#128900)
2023-06-14 [email protected] Respect allowlisted count of leaks. (flutter/flutter#128823)
2023-06-14 [email protected] Unpin flutter_plugin_android_lifecycle (flutter/flutter#128898)

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
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: 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
@gmackall gmackall mentioned this pull request Jun 20, 2023
8 tasks
auto-submit bot pushed a commit that referenced this pull request Jun 22, 2023
Unpins path_provider_android where it is pinned. Follows the same steps as #128898.

Fixes #116376
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 16, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
@gmackall gmackall deleted the integration_lockfile_2 branch January 17, 2024 04:08
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 c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Updating to androidx.annotation:annotation:1.5.0 is causing CI failures.

2 participants