Skip to content

[shared_preferences] fix cast error and mutable list error with getStringList #7355

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

Merged
merged 7 commits into from
Aug 9, 2024

Conversation

tarrinneal
Copy link
Contributor

fixes flutter/flutter#153106
fixes flutter/flutter#153112

Will add a follow up pr adding a test to the top level package after this pr lands.

@tarrinneal tarrinneal marked this pull request as ready for review August 9, 2024 19:00
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM for the version that didn't add the deliberate failure.

@tarrinneal tarrinneal added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 9, 2024
@auto-submit auto-submit bot merged commit cdd6e13 into flutter:main Aug 9, 2024
76 checks passed
@paulober
Copy link

Thanks. When can I expect the fix to release?

@stuartmorgan-g
Copy link
Contributor

Thanks. When can I expect the fix to release?

Three days ago; we use a continuous release model for the packages in this repository. You just need to flutter pub upgrade to update your transitive dependencies.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 12, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Aug 12, 2024
flutter/packages@f7b1256...d9a6de8

2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [camera]: Bump androidx.annotation:annotation from 1.8.1 to 1.8.2 in /packages/camera/camera_android/android (flutter/packages#7371)
2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [path_provider]: Bump androidx.annotation:annotation from 1.8.1 to 1.8.2 in /packages/path_provider/path_provider_android/android (flutter/packages#7376)
2024-08-12 tarrinneal@gmail.com [pigeon] removes restriction on number of custom types per file (flutter/packages#6840)
2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [pigeon]: Bump org.jetbrains.kotlin:kotlin-gradle-plugin from 2.0.0 to 2.0.10 in /packages/pigeon/platform_tests/test_plugin/android (flutter/packages#7370)
2024-08-12 fertrig@gmail.com [shared_preferences] Fixes get-all when suite name is used (flutter/packages#7335)
2024-08-12 mhvdijk@gmail.com [flutter_adaptive_scaffold] Add expanded and extra large breakpoints (flutter/packages#7300)
2024-08-12 engine-flutter-autoroll@skia.org Manual roll Flutter from b12d861 to 9b84701 (8 revisions) (flutter/packages#7366)
2024-08-10 engine-flutter-autoroll@skia.org Manual roll Flutter from 76107bd to b12d861 (14 revisions) (flutter/packages#7358)
2024-08-09 tarrinneal@gmail.com [shared_preferences] fix cast error and mutable list error with `getStringList` (flutter/packages#7355)

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-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: 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
DBowen33 pushed a commit to DBowen33/flutter that referenced this pull request Aug 16, 2024
flutter/packages@f7b1256...d9a6de8

2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [camera]: Bump androidx.annotation:annotation from 1.8.1 to 1.8.2 in /packages/camera/camera_android/android (flutter/packages#7371)
2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [path_provider]: Bump androidx.annotation:annotation from 1.8.1 to 1.8.2 in /packages/path_provider/path_provider_android/android (flutter/packages#7376)
2024-08-12 tarrinneal@gmail.com [pigeon] removes restriction on number of custom types per file (flutter/packages#6840)
2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [pigeon]: Bump org.jetbrains.kotlin:kotlin-gradle-plugin from 2.0.0 to 2.0.10 in /packages/pigeon/platform_tests/test_plugin/android (flutter/packages#7370)
2024-08-12 fertrig@gmail.com [shared_preferences] Fixes get-all when suite name is used (flutter/packages#7335)
2024-08-12 mhvdijk@gmail.com [flutter_adaptive_scaffold] Add expanded and extra large breakpoints (flutter/packages#7300)
2024-08-12 engine-flutter-autoroll@skia.org Manual roll Flutter from b12d861 to 9b84701 (8 revisions) (flutter/packages#7366)
2024-08-10 engine-flutter-autoroll@skia.org Manual roll Flutter from 76107bd to b12d861 (14 revisions) (flutter/packages#7358)
2024-08-09 tarrinneal@gmail.com [shared_preferences] fix cast error and mutable list error with `getStringList` (flutter/packages#7355)

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-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: 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
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
flutter/packages@f7b1256...d9a6de8

2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [camera]: Bump androidx.annotation:annotation from 1.8.1 to 1.8.2 in /packages/camera/camera_android/android (flutter/packages#7371)
2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [path_provider]: Bump androidx.annotation:annotation from 1.8.1 to 1.8.2 in /packages/path_provider/path_provider_android/android (flutter/packages#7376)
2024-08-12 tarrinneal@gmail.com [pigeon] removes restriction on number of custom types per file (flutter/packages#6840)
2024-08-12 49699333+dependabot[bot]@users.noreply.github.com [pigeon]: Bump org.jetbrains.kotlin:kotlin-gradle-plugin from 2.0.0 to 2.0.10 in /packages/pigeon/platform_tests/test_plugin/android (flutter/packages#7370)
2024-08-12 fertrig@gmail.com [shared_preferences] Fixes get-all when suite name is used (flutter/packages#7335)
2024-08-12 mhvdijk@gmail.com [flutter_adaptive_scaffold] Add expanded and extra large breakpoints (flutter/packages#7300)
2024-08-12 engine-flutter-autoroll@skia.org Manual roll Flutter from b12d861 to 9b84701 (8 revisions) (flutter/packages#7366)
2024-08-10 engine-flutter-autoroll@skia.org Manual roll Flutter from 76107bd to b12d861 (14 revisions) (flutter/packages#7358)
2024-08-09 tarrinneal@gmail.com [shared_preferences] fix cast error and mutable list error with `getStringList` (flutter/packages#7355)

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-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants