Skip to content

Refactor PreviewDetector initialization parameters - #178745

Merged
auto-submit[bot] merged 1 commit into
flutter-3.39-candidate.0from
i178679-project-param
Nov 18, 2025
Merged

auto-submit[bot] merged 1 commit into
flutter-3.39-candidate.0from
i178679-project-param

Conversation

@reidbaker

@reidbaker reidbaker commented Nov 18, 2025 •

Copy link
Copy Markdown
Contributor

@reidbaker
reidbaker requested a review from bkonyi November 18, 2025 17:49
@flutter-dashboard

Copy link
Copy Markdown

This pull request was opened from and to a release candidate branch. This should only be done as part of the official Flutter release process. If you are attempting to make a regular contribution to the Flutter project, please close this PR and follow the instructions at Tree Hygiene for detailed instructions on contributing to Flutter.

Reviewers: Use caution before merging pull requests to release branches. Ensure the proper procedure has been followed.

@github-actions github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Nov 18, 2025

@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 updates the initialization of PreviewDetector in a test file. The constructor call is changed to pass a FlutterProject object via the project parameter, instead of passing a Directory object via the projectRoot parameter. This change aligns with a refactoring of the PreviewDetector constructor.

@reidbaker reidbaker added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 18, 2025
@auto-submit
auto-submit Bot merged commit b6b38b4 into flutter-3.39-candidate.0 Nov 18, 2025
146 checks passed
@auto-submit
auto-submit Bot deleted the i178679-project-param branch November 18, 2025 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants