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

Stream reading breaking change #24809

Merged
merged 3 commits into from
Jul 1, 2021
Merged

Stream reading breaking change #24809

merged 3 commits into from
Jul 1, 2021

Conversation

gewarren
Copy link
Contributor

@gewarren gewarren commented Jun 23, 2021

@gewarren gewarren requested review from stephentoub and a team June 23, 2021 19:52
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

These changes LGTM @gewarren

Once you and @stephentoub agree, you can :shipit:

@BillWagner BillWagner merged commit 49404d5 into dotnet:main Jul 1, 2021
Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

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

Sorry for the delay. LGTM.

Youssef1313 pushed a commit to Youssef1313/docs that referenced this pull request Jul 5, 2021
* stream reading breaking change

* fix xrefs

* fix xrefs
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.

DeflateStream, GZipStream, and CryptoStream handling of partial and zero-byte reads
4 participants