-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[release/7.0][mono] Fix assertion while adding a new method using Hot Reload #75460
Conversation
Tagging subscribers to this area: @thaystg Issue DetailsBackport of #75432 to release/7.0 Customer ImpactWhen creating a new method and applying using hot reload it was hitting an assertion on mono runtime. TestingManual and added a unit test RiskLow risk, only reordered the variables initialization.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can add mine to make it even more official ;-)
Some of the wasm/runtime-tests are failing because the version fix from - https://github.com/dotnet/runtime/pull/75397/files#diff-dde71e6c5fa0d364c6b3a7c018b6f1a5435dbc1512757995af9c15f86674deb3R3 is needed. |
The changes here don't affect runtime-tests, IOW, the failures are unrelated. I will open a separate PR for the fix. |
|
This PR has collected the 5 sign-off Infinity Stones of the Mono team. |
Backport of #75432 to release/7.0
Customer Impact
When creating a new method and applying using hot reload it was hitting an assertion on mono runtime.
Testing
Manual and added a unit test
Risk
Low risk, only reordered the variables initialization.