-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Comparing changes
Open a pull request
base repository: flutter/packages
base: 5b1bea87e8b9
head repository: flutter/packages
compare: 3bddf2c57eed
- 5 commits
- 93 files changed
- 4 contributors
Commits on Feb 3, 2026
-
Roll Flutter from c305f1f7ad35 to bf701fefec86 (9 revisions) (#10957)
flutter/flutter@c305f1f...bf701fe 2026-02-03 engine-flutter-autoroll@skia.org Roll Dart SDK from 01ba84eeb32f to 56294a92d5cc (2 revisions) (flutter/flutter#181851) 2026-02-03 brackenavaron@gmail.com Remove material from scrollable_helpers_test.dart (flutter/flutter#181460) 2026-02-03 brackenavaron@gmail.com Reduce reliance on Material in page_transitions_test.dart (flutter/flutter#181467) 2026-02-03 116356835+AbdeMohlbi@users.noreply.github.com Fix todo about `replacing URL with constants` in `gradle_errors.dart` (flutter/flutter#181673) 2026-02-03 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from 1L4m9qCikk-JzrNWE... to UmQaaNuhkiuE8Dzug... (flutter/flutter#181841) 2026-02-03 engine-flutter-autoroll@skia.org Roll Dart SDK from 1aa8f2de7587 to 01ba84eeb32f (2 revisions) (flutter/flutter#181836) 2026-02-03 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#181738) 2026-02-03 brackenavaron@gmail.com Remove material from slivers_appbar_floating_pinned_test.dart + slivers_appbar_stretch_test.dart + sliver_persistent_header_test.dart + slivers_appbar_floating_test.dart (flutter/flutter#181469) 2026-02-02 engine-flutter-autoroll@skia.org Roll Skia from 43fa79e1c51f to f37a22506eb4 (1 revision) (flutter/flutter#181814) 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 stuartmorgan@google.com 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://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for f07e754 - Browse repository at this point
Copy the full SHA f07e754View commit details
Commits on Feb 4, 2026
-
[go_router_builder] Add support for
@TypedGoRouteParameterto custo……mize parameter names (#10793) Relates to flutter/flutter#112152 Add support for `@TypedGoRouteParameter` to customize parameter names Needs #10792 to get merged first ## Pre-Review Checklist **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
Configuration menu - View commit details
-
Copy full SHA for df85ef0 - Browse repository at this point
Copy the full SHA df85ef0View commit details -
[google_maps_flutter] Remove use of OCMock (#10863)
Eliminates usage of OCMock in the native iOS unit tests: - Creates a protocol for image asset loading that wraps that part of the registrar and the UIImage class method to replace mocking with injecting a protocol implementation. - Creates an abstract delegate for event reporting, instead of all of the sub-controllers communicating directly with the FlutterAPI instance. - Creates an abstract delegate for tile queries, instead of all of the tile controller communicating directly with the FlutterAPI instance. - Replaces mocking of our test subclass of the Google Maps view object with slightly more logic on that existing test subclass. - Replaces mocking of our existing protocols with simple manual fakes that lets us test the same assertions. - Eliminates static mock testing of GMSCameraUpdate. Unblocks flutter/flutter#119108 ## Pre-Review Checklist **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
Configuration menu - View commit details
-
Copy full SHA for f3ee98a - Browse repository at this point
Copy the full SHA f3ee98aView commit details -
[video_player] Remove OCMock (#10932)
Adds protocols to wrap usage of many AVFoundation objects in order to eliminate OCMock from the existing tests. This intentionally doesn't wrap everything, because it's already a large change; increasing test coverage in the future will almost certainly require adding more protocols. As with other PRs following this approach, in the short term this adds fair amount of production boilerplate (all the passthrough implementations in AVFVPFactory), but in the long term that boilerplate will almost all be eliminated in favor of using Swift protocol conformance in extensions, to just graft the protocols directly onto the real objects. This also requires more manual stub/mock test objects, but it does have some advantages beyond unblocking migration to Swift tests: - In many cases the tests themselves are less verbose, because some operations like triggering callbacks is cumbersome with OCMock. - It's much clearer where the test are using real AVFoundation objects that are doing things like loading network resources. (The answer is "far more than actually need to", but I deemed fixing that out of scope for this PR. This structure should make it easier to move most tests away from real objects in future test improvement work.) As an opportunistic fix while touching a lot of files anyway, I updated all the imports to the modern module import style. Helps unblock flutter/flutter#119105 ## Pre-Review Checklist - [x ] I read the [Tree Hygiene] page, which explains my responsibilities. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
Configuration menu - View commit details
-
Copy full SHA for f4ea763 - Browse repository at this point
Copy the full SHA f4ea763View commit details -
[camera_avfoundation] ios saving path (#10832)
## Description Changed iOS file storage path from `documentDirectory` to `temporaryDirectory` to ensure consistency with Android where files are stored in cache directory. ## Changes - iOS now uses `temporaryDirectory` instead of `documentDirectory` for consistency with Android ## Code changes ```swift // Before: let documentDirectory = FileManager.default.urls( for: .documentDirectory, in: .userDomainMask)[0] // After: let documentDirectory = FileManager.default.temporaryDirectory ``` ## Issues flutter/flutter#166933 ## Pre-Review Checklist **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.Configuration menu - View commit details
-
Copy full SHA for 3bddf2c - Browse repository at this point
Copy the full SHA 3bddf2cView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 5b1bea87e8b9...3bddf2c57eed