Skip to content

Handle records in documentation comments in IDE side#52737

Merged
CyrusNajmabadi merged 2 commits intodotnet:mainfrom
Youssef1313:doc-comment-records
May 2, 2021
Merged

Handle records in documentation comments in IDE side#52737
CyrusNajmabadi merged 2 commits intodotnet:mainfrom
Youssef1313:doc-comment-records

Conversation

@Youssef1313
Copy link
Member

@Youssef1313 Youssef1313 commented Apr 19, 2021

Addresses part of #52663.

The thing left for that issue is:

IDE may choose to adjust inheritance logic so that properties show some information from associated parameter

@jcouv Please add this to record structs test plan. Thanks.

@Youssef1313 Youssef1313 requested a review from a team as a code owner April 19, 2021 17:24
@ghost ghost added the Area-IDE label Apr 19, 2021
SyntaxKind.ParenthesizedLambdaExpression => ((ParenthesizedLambdaExpressionSyntax)declaration).ParameterList,
SyntaxKind.LocalFunctionStatement => ((LocalFunctionStatementSyntax)declaration).ParameterList,
SyntaxKind.AnonymousMethodExpression => ((AnonymousMethodExpressionSyntax)declaration).ParameterList,
SyntaxKind.RecordDeclaration => ((RecordDeclarationSyntax)declaration).ParameterList,
Copy link
Contributor

Choose a reason for hiding this comment

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

note to @jcouv to have this in the record-struct branch.

@jcouv jcouv mentioned this pull request Apr 19, 2021
92 tasks
@jinujoseph jinujoseph added the Community The pull request was submitted by a contributor who is not a Microsoft employee. label Apr 22, 2021
@Youssef1313
Copy link
Member Author

@CyrusNajmabadi Is this ready to merge?

@Youssef1313
Copy link
Member Author

Pinging @CyrusNajmabadi.

@Youssef1313 Youssef1313 requested a review from CyrusNajmabadi May 2, 2021 06:47
@CyrusNajmabadi CyrusNajmabadi merged commit bc98ed9 into dotnet:main May 2, 2021
@ghost ghost added this to the Next milestone May 2, 2021
@CyrusNajmabadi
Copy link
Contributor

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-IDE Community The pull request was submitted by a contributor who is not a Microsoft employee.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants