Skip to content

Conversation

@lewing
Copy link
Member

@lewing lewing commented Apr 21, 2021

sdk part in dotnet/sdk#17092

This doesn't change any behavior without the sdk change but, with the sdk change it allows the Browser build to enable runtime relink/build outside of RunAOTCompilation during publish. Enabling additional features and/or size reductions in a published app.

@ghost ghost added the area-blazor Includes: Blazor, Razor Components label Apr 21, 2021
@lewing lewing requested a review from pranavkm April 21, 2021 21:18
@lewing
Copy link
Member Author

lewing commented Apr 24, 2021

What is going on with CI in this PR?

@lewing lewing marked this pull request as ready for review April 24, 2021 01:36
@lewing lewing requested a review from a team as a code owner April 24, 2021 01:36
@SteveSandersonMS
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@SteveSandersonMS
Copy link
Member

What is going on with CI in this PR?

@lewing Looks like this PR happened during a period when CI was behaving badly. I've asked it to restart the CI run.

@pranavkm You're the main expert about this on the Blazor side, so can you approve if you're happy with it? From my point of view this looks fine and I would just approve it, but want to make sure you've had a chance to see it first.

@lewing
Copy link
Member Author

lewing commented Apr 27, 2021

Rebased to get CI green, cc @pranavkm

@lewing
Copy link
Member Author

lewing commented Apr 27, 2021

@danroth27 thoughts on the description?

Copy link
Contributor

@pranavkm pranavkm left a comment

Choose a reason for hiding this comment

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

Description is fine by me. FYI @danroth27 for input

@SteveSandersonMS
Copy link
Member

Merging to keep things moving, but we can tweak the description further later if @danroth27 has feedback.

@SteveSandersonMS SteveSandersonMS merged commit 3c91c4a into dotnet:main Apr 27, 2021
@ghost ghost added this to the 6.0-preview5 milestone Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Includes: Blazor, Razor Components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants