Skip to content

Add WebAssemblyHotReloadCapabilities msbuild property #59094

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

Open
wants to merge 1 commit into
base: release/9.0
Choose a base branch
from

Conversation

tmat
Copy link
Member

@tmat tmat commented Nov 21, 2024

Enables Hot Reload to detect capabilities before WASM runtime is loaded into the browser.

Contributes to a fix of #59027

Enables Hot Reload to detect capabilities before WASM runtime is loaded into the browser.
@tmat tmat requested a review from a team as a code owner November 21, 2024 18:44
@ghost ghost added the area-blazor Includes: Blazor, Razor Components label Nov 21, 2024
@tmat
Copy link
Member Author

tmat commented Nov 21, 2024

@MackinnonBuck @javiercn ptal

@dotnet-policy-service dotnet-policy-service bot added this to the 9.0.x milestone Nov 21, 2024
@wtgodbe
Copy link
Member

wtgodbe commented Nov 27, 2024

@tmat can you ask tactics for approval on this one?

@dotnet-policy-service dotnet-policy-service bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Dec 5, 2024
@lewing
Copy link
Member

lewing commented Jan 22, 2025

Is this still needed?

@tmat
Copy link
Member Author

tmat commented Jan 22, 2025

Yes, although it's currently unclear which branch it should go to.

@tmat tmat changed the base branch from release/9.0 to main January 25, 2025 00:21
@tmat tmat changed the base branch from main to release/9.0 January 25, 2025 00:21
@javiercn
Copy link
Member

@tmat is this needed in the 9.0 release?

@tmat
Copy link
Member Author

tmat commented Feb 1, 2025

We can move it to main.

@@ -2,6 +2,7 @@
<PropertyGroup>
<BlazorWebAssemblyJSPath>$(MSBuildThisFileDirectory)blazor.webassembly.js</BlazorWebAssemblyJSPath>
<BlazorRoutingEnableRegexConstraint Condition="'$(BlazorRoutingEnableRegexConstraint)' == ''">false</BlazorRoutingEnableRegexConstraint>
<WebAssemblyHotReloadCapabilities>Baseline;AddMethodToExistingType;AddStaticFieldToExistingType;NewTypeDefinition;ChangeCustomAttributes;AddInstanceFieldToExistingType;GenericAddMethodToExistingType;GenericUpdateMethod;UpdateParameters;GenericAddFieldToExistingType</WebAssemblyHotReloadCapabilities>
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this go to the underlaying WebAssembly SDK? It's common to all ".NET in browser", not specific to Blazor, right?

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 pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants