Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update File.cs #85648

Merged
merged 1 commit into from
May 9, 2023
Merged

Update File.cs #85648

merged 1 commit into from
May 9, 2023

Conversation

SiqiLu
Copy link
Contributor

@SiqiLu SiqiLu commented May 2, 2023

I looked up the documentation (document link: https://learn.microsoft.com/en-us/dotnet/api/system.io.file.exists?view=net-7.0) and it states that "If path describes a directory, this method returns false." Additionally, after testing it myself, I found that the comment here should be corrected.

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label May 2, 2023
@ghost
Copy link

ghost commented May 2, 2023

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

Issue Details

I looked up the documentation (document link: https://learn.microsoft.com/en-us/dotnet/api/system.io.file.exists?view=net-7.0) and it states that "If path describes a directory, this method returns false." Additionally, after testing it myself, I found that the comment here should be corrected.

Author: SiqiLu
Assignees: -
Labels:

area-System.IO, community-contribution

Milestone: -

@SiqiLu
Copy link
Contributor Author

SiqiLu commented May 2, 2023

@dotnet-policy-service agree

@adamsitnik
Copy link
Member

The failures are unrelated. Thank you for your contribution @SiqiLu !

@adamsitnik adamsitnik merged commit 8c96a18 into dotnet:main May 9, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jun 8, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
area-System.IO community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants