-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Doc fixes for .NET 10 Preview 1 update #113027
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Overview
This PR updates XML documentation comments for .NET 10 Preview 1 by replacing tags with tags and by standardizing boolean return values using and for consistency.
- Updated XML documentation in PhysicalFileProvider.cs
- Standardized boolean documentation in various AsyncEnumerable files
Reviewed Changes
File | Description |
---|---|
src/libraries/Microsoft.Extensions.FileProviders.Physical/src/PhysicalFileProvider.cs | Replaced with in doc comments for clarity. |
src/libraries/System.Linq.AsyncEnumerable/src/System/Linq/AnyAsync.cs | Updated boolean return documentation markup. |
src/libraries/System.Linq.AsyncEnumerable/src/System/Linq/AllAsync.cs | Updated boolean return documentation markup. |
src/libraries/System.Linq.AsyncEnumerable/src/System/Linq/ContainsAsync.cs | Updated boolean return documentation markup. |
src/libraries/System.Linq.AsyncEnumerable/src/System/Linq/SequenceEqualAsync.cs | Updated boolean return documentation markup. |
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume these were fixed in source too?
This is the source. I didn't fix in the API docs repo since I assume the XML file is refreshed for each preview? |
Microsoft.Extensions have triple slash comments as source of truth.
This is confusing me 😅 |
See dotnet/dotnet-api-docs#11032 build report.