Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@robert-ancell
Copy link
Contributor

As requested in #16977 to match #17210

@auto-assign auto-assign bot requested a review from cbracken March 24, 2020 21:48
@stuartmorgan-g stuartmorgan-g self-requested a review March 24, 2020 22:31
@robert-ancell robert-ancell force-pushed the linux-shell-new-project-layout branch 3 times, most recently from 9fd9634 to 873a357 Compare March 25, 2020 03:29
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

Looks great! Just a couple nits.

@robert-ancell robert-ancell force-pushed the linux-shell-new-project-layout branch from 873a357 to a9b07e4 Compare March 25, 2020 20:45
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM!

The CI failures are clearly unrelated to the PR; we can land this once the tree is open.

@stuartmorgan-g stuartmorgan-g merged commit 6befd4a into flutter:master Mar 31, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 1, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 1, 2020
@robert-ancell robert-ancell deleted the linux-shell-new-project-layout branch April 8, 2020 00:20
goderbauer pushed a commit to goderbauer/engine that referenced this pull request Apr 16, 2020
Matches the structure used on Windows of assuming a directory format.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants