Skip to content

Conversation

@kenzieschmoll
Copy link
Member

@kenzieschmoll kenzieschmoll commented May 30, 2024

This is a reland of #7599, which was reverted in #7688 because the new bootstrapping syntax is not supported in google3. This is still true, but I have prepared cl/638797386 to use the existing bootstrapping in google3 until the new syntax is supported.

Work toward #7856.

@kenzieschmoll kenzieschmoll requested a review from a team as a code owner May 30, 2024 22:43
@kenzieschmoll kenzieschmoll requested review from bkonyi, elliette and eyebrowsoffire and removed request for a team May 30, 2024 22:43
}

// Bootstrap app for 1P environments:
function bootstrapAppFor1P() {
Copy link
Member

@elliette elliette May 30, 2024

Choose a reason for hiding this comment

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

Do we need this with cl/638797386?

Copy link
Member Author

Choose a reason for hiding this comment

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

I would opt to leave it so that things can work without needing to change anything once b/343786318 is resolved.

Copy link
Member

@elliette elliette left a comment

Choose a reason for hiding this comment

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

LGTM with question

@kenzieschmoll kenzieschmoll merged commit c619d5d into flutter:master May 31, 2024
@kenzieschmoll kenzieschmoll deleted the wasm-bootstrap branch May 31, 2024 15:49
@kenzieschmoll kenzieschmoll mentioned this pull request May 31, 2024
13 tasks
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