Skip to content

[camera] Updates min SDK to 3.29#9822

Closed
stuartmorgan-g wants to merge 5 commits intoflutter:mainfrom
stuartmorgan-g:new-formatter-camera
Closed

[camera] Updates min SDK to 3.29#9822
stuartmorgan-g wants to merge 5 commits intoflutter:mainfrom
stuartmorgan-g:new-formatter-camera

Conversation

@stuartmorgan-g
Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g commented Aug 15, 2025

For the camera packages:

  • Updates the min SDK version to 3.29
  • Runs the autoformatter with the new format
  • Update code excerpts

This does not update versions because pushing format changes (even in READMEs) isn't worth doing a release that drops an SDK version (which we don't normally version)

Prep for #9816

Copy link

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

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 the minimum SDK version for the camera packages to Flutter 3.29/Dart 3.7 and applies auto-formatting across many files. The changes are mostly consistent with the goal of updating dependencies and code style. However, there's a discrepancy between the PR title, which mentions SDK 3.27, and the actual changes that update to 3.29. It would be beneficial to align the PR title and description with the code changes. I've also left a couple of comments on duplicated lines in the changelog files that could be cleaned up.

@stuartmorgan-g
Copy link
Collaborator Author

I've also left a couple of comments on duplicated lines in the changelog files that could be cleaned up.

Oops, forgot that step on this PR. Fixed. Thanks Gemini!

@stuartmorgan-g stuartmorgan-g changed the title [camera] Updates min SDK to 3.27 [camera] Updates min SDK to 3.29 Aug 15, 2025
@stuartmorgan-g
Copy link
Collaborator Author

Obsoleted by the combination PR.

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.

1 participant