Skip to content

Roll pub packages#185172

Closed
flutter-pub-roller-bot wants to merge 1 commit into
flutter:masterfrom
flutter-pub-roller-bot:packages-autoroller-branch-301
Closed

Roll pub packages#185172
flutter-pub-roller-bot wants to merge 1 commit into
flutter:masterfrom
flutter-pub-roller-bot:packages-autoroller-branch-301

Conversation

@flutter-pub-roller-bot

@flutter-pub-roller-bot flutter-pub-roller-bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

This PR was generated by flutter update-packages --force-upgrade.

@flutter-pub-roller-bot flutter-pub-roller-bot added tool Affects the "flutter" command-line tool. See also t: labels. autosubmit Merge PR when tree becomes green via auto submit App labels Apr 16, 2026
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Apr 16, 2026
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Apr 16, 2026
View reviewed changes

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates several dependencies across the repository, including vm_service to 15.1.0, googleapis_auth to 2.3.0, hooks_runner to 1.2.1, and webview_flutter_wkwebview to 3.24.3. It also introduces google_cloud as a transitive dependency and updates relevant pubspec checksums. I have no feedback to provide.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 16, 2026
@auto-submit

auto-submit Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/185172, because - The status or check suite Mac tool_tests_general has failed. Please fix the issues identified (or deflake) before re-applying this label.

@DanTup DanTup mentioned this pull request Apr 17, 2026
Closed
@DanTup

DanTup commented Apr 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This is failing due to a change in hooks_runner that added a new method to a base abstract class. I'm looking at whether we can just add a => throw UnimplementException() in the related test class.

Edit: #185274

DanTup added a commit to DanTup/flutter that referenced this pull request Apr 20, 2026
@DanTup
Roll pub packages
401b082 
This PR was generated by `flutter update-packages --force-upgrade`, with a small change to add a `toJson()` method to a fake test class because `hooks_runner` added this to the base abstract class (which caused the bots roll at flutter#185172 to fail).

Closes flutter#185172
@DanTup DanTup mentioned this pull request Apr 20, 2026
Merged
@Piinks

Piinks commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Fixed by #184880
Closing so the bot can create a fresh pull

@Piinks Piinks closed this Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fluttergithubbot fluttergithubbot fluttergithubbot approved these changes
+1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

CICD Run CI/CD tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@flutter-pub-roller-bot @DanTup @Piinks @fluttergithubbot