Skip to content

[release/7.0] [hot_reload] implement param reflection - #77734

Merged
carlossanlop merged 15 commits into
release/7.0from
backport/pr-77563-to-release/7.0
Nov 3, 2022
Merged

[release/7.0] [hot_reload] implement param reflection#77734
carlossanlop merged 15 commits into
release/7.0from
backport/pr-77563-to-release/7.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Nov 1, 2022

Copy link
Copy Markdown
Contributor

Backport of #77563 to release/7.0

/cc @lambdageek

Customer Impact

In .NET 7 we enabled mobile and WebAssembly users to use Hot Reload to add new functions to existing classes. In MAUI, this allows users to use XAML Hot Reload and C# Hot Reload together to create new elements and attach new event handlers to them in a running hot reload session. This PR fixes a crash when MAUI called the new added callbacks using reflection.

This PR enables a key scenario.

Testing

New CI tests. Manual verification that the PR addresses the MAUI crashes.

Risk

Low. All the new code is gated on a hot reload change being applied to the runtime. In other words, production apps will not run any of the modified code paths.

@ghostghost added the area-EnC-mono Hot Reload for WebAssembly, iOS/Android, etc label Nov 1, 2022
@lambdageeklambdageek added the Servicing-consider Issue for next servicing release review label Nov 1, 2022
@lambdageeklambdageek added this to the 7.0.x milestone Nov 1, 2022
@marek-safarmarek-safar added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 2, 2022
@carlossanlop

Copy link
Copy Markdown
Contributor

Approved by Tactics. Signed off by area owner. No OOB package authoring changes needed.
CI failures is an unrelated WASM issue that was just fixed here: #76939 so there's no need to restart the CI.
Ready to merge. :shipit:

@carlossanlop
carlossanlop merged commit 9ef4b1e into release/7.0Nov 3, 2022
@carlossanlop
carlossanlop deleted the backport/pr-77563-to-release/7.0 branch November 3, 2022 00:56
@ghostghost locked as resolved and limited conversation to collaborators Dec 3, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-EnC-monoHot Reload for WebAssembly, iOS/Android, etcServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@carlossanlop@vargaz@marek-safar@lambdageek