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

.Net: Removed System.Memory.Data dependency #9970

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dmytrostruk
Copy link
Member

Motivation and Context

Removed unused System.Memory.Data dependency from the solution.

Contribution Checklist

@dmytrostruk dmytrostruk self-assigned this Dec 13, 2024
@dmytrostruk dmytrostruk requested a review from a team as a code owner December 13, 2024 15:31
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel labels Dec 13, 2024
@@ -64,7 +64,7 @@ public void ItProvideStatusForResponsesWithoutContent()
// Assert
Assert.NotNull(httpOperationException);
Assert.NotNull(httpOperationException.StatusCode);
Assert.Empty(httpOperationException.ResponseContent!);
Assert.Null(httpOperationException.ResponseContent);
Copy link
Member

Choose a reason for hiding this comment

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

I wonder how removal of the package dependency influenced the property value?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants