Skip to content

[ci] Update minimums for 3.13 stable#4731

Merged
auto-submit[bot] merged 5 commits intoflutter:mainfrom
stuartmorgan-g:stable-3-13-update-mins
Aug 17, 2023
Merged

[ci] Update minimums for 3.13 stable#4731
auto-submit[bot] merged 5 commits intoflutter:mainfrom
stuartmorgan-g:stable-3-13-update-mins

Conversation

@stuartmorgan-g
Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g commented Aug 17, 2023

  • Updates the N-1 and N-2 analysis bot configs to 3.7 and 3.10
  • Updates the minimum allowed SDK to 3.7, since that's now the earliest we have any testing for.
  • Updates all packages to Flutter 3.7/Dart 2.19 as a minimum.
    • In packages where we hadn't released since the last minimum version bump, I removed the previous line from the #NEXT section to make it less confusing when we do release.

Per https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#version this does not update any package versions.

@stuartmorgan-g stuartmorgan-g added the override: no versioning needed Override the check requiring version bumps for most changes label Aug 17, 2023
@stuartmorgan-g stuartmorgan-g changed the title [ci] Update minimums for new stable [ci] Update minimums for 3.13 stable Aug 17, 2023
jonahwilliams
jonahwilliams approved these changes Aug 17, 2023
View reviewed changes
Copy link
Contributor

@jonahwilliams jonahwilliams 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-g
Copy link
Collaborator Author

stuartmorgan-g commented Aug 17, 2023

Reviewers: only @tarrinneal and/or @bparrishMines need to review this.

chunhtai
chunhtai approved these changes Aug 17, 2023
View reviewed changes
Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

tarrinneal
tarrinneal approved these changes Aug 17, 2023
View reviewed changes
Copy link
Contributor

@tarrinneal tarrinneal left a comment

Choose a reason for hiding this comment

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

that's a lot of nums

bparrishMines
bparrishMines approved these changes Aug 17, 2023
View reviewed changes
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

@LongCatIsLooong LongCatIsLooong mentioned this pull request Aug 17, 2023
Merged
11 tasks
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Aug 18, 2023
Merged
This was referenced Aug 18, 2023
Closed
Merged
This was referenced Sep 5, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bparrishMines bparrishMines bparrishMines approved these changes

@chunhtai chunhtai chunhtai approved these changes

@tarrinneal tarrinneal tarrinneal approved these changes

@cbracken cbracken Awaiting requested review from cbracken

@cyanglaz cyanglaz Awaiting requested review from cyanglaz

@hellohuanlin hellohuanlin Awaiting requested review from hellohuanlin hellohuanlin is a code owner

@LouiseHsu LouiseHsu Awaiting requested review from LouiseHsu LouiseHsu is a code owner

@vashworth vashworth Awaiting requested review from vashworth vashworth is a code owner

@camsim99 camsim99 Awaiting requested review from camsim99 camsim99 is a code owner

@gmackall gmackall Awaiting requested review from gmackall gmackall is a code owner

@reidbaker reidbaker Awaiting requested review from reidbaker reidbaker is a code owner

@ditman ditman Awaiting requested review from ditman ditman is a code owner

@MitchellGoodwin MitchellGoodwin Awaiting requested review from MitchellGoodwin

@yjbanov yjbanov Awaiting requested review from yjbanov yjbanov is a code owner

@Hixie Hixie Awaiting requested review from Hixie Hixie is a code owner

@gspencergoog gspencergoog Awaiting requested review from gspencergoog

@dnfield dnfield Awaiting requested review from dnfield

@keyonghan keyonghan Awaiting requested review from keyonghan

@domesticmouse domesticmouse Awaiting requested review from domesticmouse

@goderbauer goderbauer Awaiting requested review from goderbauer

@Piinks Piinks Awaiting requested review from Piinks

+1 more reviewer

@jonahwilliams jonahwilliams jonahwilliams approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

autosubmit Merge PR when tree becomes green via auto submit App override: no versioning needed Override the check requiring version bumps for most changes p: animations p: camera p: cross_file p: css_colors p: cupertino_icons p: dynamic_layouts The dynamic_layouts package p: espresso p: extension_google_sign_in_as_googleapis_auth p: file_selector p: flutter_adaptive_scaffold p: flutter_image p: flutter_lints p: flutter_markdown p: flutter_migrate p: flutter_plugin_android_lifecycle p: flutter_template_images p: go_router_builder p: go_router p: google_maps_flutter p: google_sign_in p: image_picker p: in_app_purchase p: ios_platform_images p: local_auth p: metrics_center p: multicast_dns p: palette_generator p: path_provider p: pigeon p: plugin_platform_interface p: pointer_interceptor p: quick_actions p: rfw Remote Flutter Widgets p: shared_preferences p: standard_message_codec p: url_launcher p: video_player p: web_benchmarks p: webview_flutter p: xdg_directories platform-android platform-ios platform-linux platform-macos platform-web platform-windows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@stuartmorgan-g @jonahwilliams @bparrishMines @chunhtai @tarrinneal