Skip to content

IOperation: expose information about the type parameter that is going to be used as the ".constrained" type for invoking abstract static methods#61811

Merged
AlekseyTs merged 2 commits intodotnet:mainfrom
AlekseyTs:Issue53803
Jun 27, 2022

Conversation

@AlekseyTs
Copy link
Copy Markdown
Contributor

Closes #53803.

Copy link
Copy Markdown
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.

Done review pass (commit 3). The general API shape seems fine to me, pending API review.

Copy link
Copy Markdown
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.

LGTM (commit 5), pending API approval

@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler For the second review. The API got approved.

@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler For the second review.

@RikkiGibson RikkiGibson self-assigned this Jun 21, 2022
@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler For the second review.

1 similar comment
@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@RikkiGibson, @dotnet/roslyn-compiler For the second review.

@AlekseyTs AlekseyTs enabled auto-merge June 23, 2022 01:25
@AlekseyTs AlekseyTs disabled auto-merge June 27, 2022 16:23
@AlekseyTs AlekseyTs merged commit 8340574 into dotnet:main Jun 27, 2022
@ghost ghost added this to the Next milestone Jun 27, 2022
@RikkiGibson RikkiGibson modified the milestones: Next, 17.3 P3 Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature - StaticAbstractMembersInInterfaces

Projects

None yet

3 participants