Skip to content
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

[hot_reload] ignore modified MONO_TABLE_TYPEDEF rows in update #90166

Merged
merged 4 commits into from
Aug 9, 2023

Conversation

lambdageek
Copy link
Member

We may want to validate that Parent, Interfaces and Attributes columns haven't changed, but it's tricky and might be overly restrictive.

Fixes #90037

We may want to validate that Parent, Interfaces and Attributes columns
haven't changed, but it's tricky and might be overly restrictive
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-EnC-mono Hot Reload for WebAssembly, iOS/Android, etc label Aug 8, 2023
@ghost ghost assigned lambdageek Aug 8, 2023
@lambdageek

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@lambdageek
Copy link
Member Author

/backport to release/7.0-staging

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Started backporting to release/7.0-staging: https://github.com/dotnet/runtime/actions/runs/5800183196

src/mono/mono/component/hot_reload.c Outdated Show resolved Hide resolved
It's just two branches with comments. Remove the `if` entirely
@lambdageek lambdageek merged commit 88633ae into dotnet:main Aug 9, 2023
136 of 138 checks passed
@ghost ghost locked as resolved and limited conversation to collaborators Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-EnC-mono Hot Reload for WebAssembly, iOS/Android, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WebToolsException when using hot reload to apply change to Blazor WebAssembly app
2 participants