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

Update PipeOptions.xml #8693

Merged
merged 1 commit into from
Dec 6, 2022
Merged

Update PipeOptions.xml #8693

merged 1 commit into from
Dec 6, 2022

Conversation

halter73
Copy link
Member

@halter73 halter73 commented Dec 6, 2022

Summary

Update PipeOptions.PauseWriterThreshold property docs to include detail about unlimited buffering that's already on the parameter to the constructor. See https://learn.microsoft.com/en-us/dotnet/api/system.io.pipelines.pipeoptions.-ctor?view=dotnet-plat-ext-7.0

@carlossanlop

@ghost
Copy link

ghost commented Dec 6, 2022

Tagging subscribers to this area: @dotnet/area-system-io
See info in area-owners.md if you want to be subscribed.

Issue Details

Summary

Update PipeOptions.PauseWriterThreshold property docs to include detail about unlimited buffering that's already on the parameter to the constructor. See https://learn.microsoft.com/en-us/dotnet/api/system.io.pipelines.pipeoptions.-ctor?view=dotnet-plat-ext-7.0

@carlossanlop

Author: halter73
Assignees: halter73
Labels:

area-System.IO

Milestone: -

@opbld34
Copy link

opbld34 commented Dec 6, 2022

Learn Build status updates of commit 01e7a49:

✅ Validation status: passed

File Status Preview URL Details
xml/System.IO.Pipelines/PipeOptions.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@halter73 halter73 merged commit ad7db07 into main Dec 6, 2022
@halter73 halter73 deleted the halter73/doc-non-blocking-pipe branch December 6, 2022 22:34
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.

4 participants