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

Revise usage of <code> tag in IOperation docs #74719

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

DoctorKrolic
Copy link
Contributor

Noticed, that in all places in IOperation doc <code> tag was used incorrectly. In many cases <see langword... should be used instead. In several places inline code (<c>) looks a lot better

Example:

Before:
vjpm3C5pUl

After:
0wfaz8IIH5

@DoctorKrolic DoctorKrolic requested a review from a team as a code owner August 12, 2024 16:42
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Compilers untriaged Issues and PRs which have not yet been triaged by a lead labels Aug 12, 2024
@dotnet-policy-service dotnet-policy-service bot added the Community The pull request was submitted by a contributor who is not a Microsoft employee. label Aug 12, 2024
@@ -2975,8 +2975,8 @@ public interface IRecursivePatternOperation : IPatternOperation
/// </summary>
ITypeSymbol MatchedType { get; }
/// <summary>
/// The symbol, if any, used for the fetching values for subpatterns. This is either a <code>Deconstruct</code>
/// method, the type <code>System.Runtime.CompilerServices.ITuple</code>, or null (for example, in
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this just be a cref?

should the null be a langword?

Copy link
Member

@333fred 333fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dotnet/roslyn-compiler for a second review.

@333fred 333fred enabled auto-merge (squash) August 12, 2024 17:40
@DoctorKrolic
Copy link
Contributor Author

Can someone restart failed CI pipelines please? Or just straight up merge, there are no code changes anyway

@333fred 333fred merged commit 970b178 into dotnet:main Aug 12, 2024
24 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Aug 12, 2024
@DoctorKrolic DoctorKrolic deleted the ioperation-docs-code branch August 12, 2024 19:00
@dibarbet dibarbet modified the milestones: Next, 17.12 P2 Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Compilers Community The pull request was submitted by a contributor who is not a Microsoft employee. untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants