Skip to content

Set some new Wasm properties so the Sdk can use Workloads when available #32036

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 27, 2021

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