-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Explicitly use para separators in the System.Runtime.Intrinsics XML docs #106652
Conversation
This is a fairly mechanical change and was done via a general search/replace |
Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics |
@@ -8,150 +8,150 @@ namespace System.Runtime.Intrinsics.Arm | |||
public enum SveMaskPattern : byte | |||
{ | |||
/// <summary> | |||
/// POW2 | |||
/// <para>POW2</para> |
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 don't think it's needed for single lines, but might be OK anyway.
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.
In this case it was more about getting the bulk of it correct without needing to manually audit all the lines.
I'll see if I can recognize the single line case and fix it up to not include the tag...
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.
Was able to construct a regex that handled the scenario here, should no longer have <para>
for single line summaries
c86c624
to
dddcdc8
Compare
CC. @carlossanlop, @gewarren