Skip to content

[flutter_tool] initialize flutter root in executable#69549

Merged
jonahwilliams merged 2 commits intoflutter:masterfrom
jonahwilliams:remove-flutter-root
Nov 5, 2020
Merged

[flutter_tool] initialize flutter root in executable#69549
jonahwilliams merged 2 commits intoflutter:masterfrom
jonahwilliams:remove-flutter-root

Conversation

@jonahwilliams
Copy link
Contributor

@jonahwilliams jonahwilliams commented Nov 2, 2020

Description

initialize the flutter root as early as possible. This allows all interfaces to be normally injected into commands, even if those objects might read from the flutter root.

@flutter-dashboard flutter-dashboard bot added c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. work in progress; do not review labels Nov 2, 2020
@google-cla google-cla bot added the cla: yes label Nov 2, 2020
@jonahwilliams jonahwilliams marked this pull request as ready for review November 2, 2020 16:10
Copy link
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

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

LGTM, let's see if anything breaks!

@jonahwilliams jonahwilliams merged commit dd8820b into flutter:master Nov 5, 2020
@jonahwilliams jonahwilliams deleted the remove-flutter-root branch November 5, 2020 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants