Skip to content

Conversation

michaelnebel
Copy link
Contributor

No description provided.

@michaelnebel michaelnebel force-pushed the csharp/test-clearscontent branch from a605979 to ddde1d4 Compare May 22, 2022 13:17
@michaelnebel michaelnebel changed the title C#: Remove default clears content (only used for a DCA test). C#: Remove default clears content. May 22, 2022
…, we need to explicitly say that fields are cleared.
@michaelnebel
Copy link
Contributor Author

DCA doesn't show any performance degradation or any change in analysis behavior.
@hvirved : Carving this out of the Dotnet Runtime models PR.

@michaelnebel michaelnebel marked this pull request as ready for review May 23, 2022 07:02
@michaelnebel michaelnebel requested review from a team as code owners May 23, 2022 07:02
@michaelnebel michaelnebel added the no-change-note-required This PR does not need a change note label May 23, 2022
@michaelnebel
Copy link
Contributor Author

The failing integration tests are unrelated to the changes in this PR. Will merge.

@michaelnebel michaelnebel merged commit bf958ff into github:main May 23, 2022
@michaelnebel michaelnebel deleted the csharp/test-clearscontent branch November 18, 2022 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants