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

[release/6.0-preview6] [FileStream] add tests for device and UNC paths #54750

Merged

Conversation

github-actions[bot]
Copy link
Contributor

Backport of #54545 to release/6.0-preview6

/cc @danmoseley @adamsitnik

Customer Impact

Testing

Risk

@ghost
Copy link

ghost commented Jun 25, 2021

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

Issue Details

Backport of #54545 to release/6.0-preview6

/cc @danmoseley @adamsitnik

Customer Impact

Testing

Risk

Author: github-actions[bot]
Assignees: -
Labels:

area-System.IO

Milestone: -

@danmoseley
Copy link
Member

Porting this so at least the tests run. We can choose whether or not to merge.

@danmoseley
Copy link
Member

Added fix for fd leak, running again.

@danmoseley
Copy link
Member

Tests only as previously discussed. Does not need to flow.

@safern safern merged commit d4baa2b into release/6.0-preview6 Jun 26, 2021
@safern safern deleted the backport/pr-54545-to-release/6.0-preview6 branch June 26, 2021 01:29
@ghost ghost locked as resolved and limited conversation to collaborators Jul 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants