Skip to content

Delete Provider placeholder screen. - #10046

Merged
kenzieschmoll merged 4 commits into
flutter:masterfrom
kenzieschmoll:provider-screen
Oct 20, 2026
Merged

kenzieschmoll merged 4 commits into
flutter:masterfrom
kenzieschmoll:provider-screen

Conversation

@kenzieschmoll

Copy link
Copy Markdown
Member

This screen is no longer needed. The provider screen has been reimplemented as a DevTools extension for ~2 years now.

@kenzieschmoll
kenzieschmoll requested a review from a team as a code owner October 19, 2026 21:25
@kenzieschmoll
kenzieschmoll requested review from srawlins and removed request for a team October 19, 2026 21:25

@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 removes the legacy ProviderScreen from DevTools, as it has been migrated to a DevTools extension. This involves deleting the provider_screen.dart file and removing its associated imports, exports, and registration in app.dart. The review feedback points out that removing ProviderScreen makes the special-casing filtering logic in the _screens getter of app.dart redundant, presenting an opportunity to simplify the code.

Comment thread packages/devtools_app/lib/src/app.dart
@kenzieschmoll
kenzieschmoll merged commit 94adff0 into flutter:master Oct 20, 2026
51 checks passed
@kenzieschmoll
kenzieschmoll deleted the provider-screen branch October 20, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants