Skip to content

Upgrade flutter candidate and DCM version - #10048

Open
kenzieschmoll wants to merge 7 commits into
flutter:masterfrom
kenzieschmoll:upgrades
Open

kenzieschmoll wants to merge 7 commits into
flutter:masterfrom
kenzieschmoll:upgrades

Conversation

@kenzieschmoll

Copy link
Copy Markdown
Member

Upgrade flutter version and DCM version and fix associated lints and warnings.

@kenzieschmoll
kenzieschmoll requested review from a team and srawlins as code owners October 20, 2026 16:31

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

Copy link
Copy Markdown
Contributor

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 cleans up redundant argument values across several DevTools packages. Specifically, it simplifies CrossAxisAlignment.values.toList(growable: true) to toList(), replaces Expanded(child: SizedBox()) with Spacer(), and updates numerous Completer.complete(null) calls to complete(). It also updates the changelog for devtools_app_shared and updates package dependencies in pubspec.lock. I have no feedback to provide.

@kenzieschmoll

Copy link
Copy Markdown
Member Author

/gemini review

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

Copy link
Copy Markdown
Contributor

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 cleans up redundant null arguments in Completer.complete() calls, replaces an empty Expanded box with Spacer, and introduces custom SIMD formatting in variable_factory.dart to ensure consistent display representation across VM and web platforms. Feedback suggests using lane.toUnsigned(32) instead of bitwise AND to guarantee correct unsigned 32-bit hex formatting on the web, and adjusting the CHANGELOG entry to use past tense and end with a period.

Comment thread packages/devtools_app_shared/CHANGELOG.md Outdated
Comment thread packages/devtools_app/lib/src/shared/diagnostics/variable_factory.dart Outdated
kenzieschmoll and others added 2 commits October 20, 2026 11:51
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…ory.dart

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@kenzieschmoll
kenzieschmoll requested a review from bkonyi October 20, 2026 18:51

This branch has not been deployed

No deployments
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.

2 participants