Skip to content
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

[various] Update example apps to Kotlin 1.9.0 #7998

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

stuartmorgan
Copy link
Contributor

Follow-up to #7997 to update all example apps to 1.9.0 to protect us from similar OOB failures as other transitive dependencies in the ecosystem update to newer versions of Kotlin.

Follow-up to flutter#7997 to update all
example apps to 1.9.0 to protect us from similar OOB failures as other
transitive dependencies in the ecosystem update to newer versions of
Kotlin.
@stuartmorgan
Copy link
Contributor Author

stuartmorgan commented Nov 1, 2024

@gmackall / @reidbaker My reasoning here is that now that we have a Flutter Fix telling people what to do in cases like #7997, and that we deliberately removed the workaround that was intended to prevent it in #7088, we consider that kind of failure something clients should fix in their app (per the Flutter Fix) rather than a bug in our plugins. If that's the case, it's better to preemptively update our example apps to avoid hitting it as an OOB failure.

If those assumptions aren't correct (and, e.g., we want to revert that url_launcher_android dependabot change because it pulled in Kotlin 1.9.0), let me know and we can revisit our testing approach instead.

Copy link
Member

@gmackall gmackall left a comment

Choose a reason for hiding this comment

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

@gmackall / @reidbaker My reasoning here is that now that we have a Flutter Fix telling people what to do in cases like #7997, and that we deliberately removed the workaround that was intended to prevent it in #7088, we consider that kind of failure something clients should fix in their app (per the Flutter Fix) rather than a bug in our plugins. If that's the case, it's better to preemptively update our example apps to avoid hitting it as an OOB failure.

I think it makes sense to have users update now that we have an error handler in place, lgtm.

Copy link
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

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

LGTM

@stuartmorgan stuartmorgan added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 4, 2024
Copy link
Contributor

auto-submit bot commented Nov 4, 2024

auto label is removed for flutter/packages/7998, due to Pull request flutter/packages/7998 is not in a mergeable state.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 4, 2024
@stuartmorgan stuartmorgan added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 4, 2024
@auto-submit auto-submit bot merged commit abadfb5 into flutter:main Nov 4, 2024
76 checks passed
@stuartmorgan stuartmorgan deleted the android-examples-kotlin-1-9 branch November 4, 2024 23:48
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 5, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 5, 2024
flutter/packages@796afa3...7219431

2024-11-05 [email protected] [vector_graphics*] Relax dependency constraints of vector_graphics, vector_graphics_codec, vector_graphics_compiler, flutter_svg  (flutter/packages#8018)
2024-11-04 [email protected] [various] Add `missing_code_block_language_in_doc_comment` lint to flutter/packages. (flutter/packages#6473)
2024-11-04 [email protected] [various] Update example apps to Kotlin 1.9.0 (flutter/packages#7998)
2024-11-04 [email protected] [go_router] add current state getter (flutter/packages#7651)
2024-11-04 [email protected] Applied Gradle Plugins Declaratively for Multiple Plugin Example Apps (flutter/packages#7968)
2024-11-04 [email protected] Roll Flutter from f86b777 to 8591d0c (16 revisions) (flutter/packages#8015)
2024-11-04 [email protected] [camera_windows] Revert: Support image streams on Windows platform (flutter/packages#7951)
2024-11-02 [email protected] [camera] Use Pigeon for Windows C++->Dart (flutter/packages#8001)
2024-11-02 [email protected] [script/tool] update dependencies (flutter/packages#7992)
2024-11-01 [email protected] Roll Flutter from 0fe6153 to f86b777 (16 revisions) (flutter/packages#8000)
2024-11-01 [email protected] [path_parsing] deprecate utility functions that should be private (flutter/packages#7993)

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 [email protected] 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
Development

Successfully merging this pull request may close these issues.

6 participants