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.

Sign up for GitHub

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

Jump to bottom

Add support for implementations of interface static abstract user-defined conversions in classes and structures #53529

Merged
merged 1 commit into from
May 21, 2021
Merged

Add support for implementations of interface static abstract user-defined conversions in classes and structures #53529

merged 1 commit into from
May 21, 2021

Conversation

AlekseyTs
Copy link
Contributor

No description provided.

@AlekseyTs AlekseyTs requested review from 333fred, RikkiGibson and a team May 19, 2021 22:11
@RikkiGibson RikkiGibson self-assigned this May 19, 2021
@AlekseyTs
Copy link
Contributor Author

@333fred, @RikkiGibson, @dotnet/roslyn-compiler Please review.

333fred
333fred approved these changes May 20, 2021
View reviewed changes
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.

LGTM (commit 1)

@AlekseyTs
Copy link
Contributor Author

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

@AlekseyTs
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@AlekseyTs AlekseyTs merged commit 766afd9 into dotnet:features/StaticAbstractMembersInInterfaces May 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@333fred 333fred 333fred approved these changes

@RikkiGibson RikkiGibson RikkiGibson approved these changes

Assignees

@RikkiGibson RikkiGibson

Labels
Area-Compilers New Feature - StaticAbstractMembersInInterfaces
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AlekseyTs @333fred @RikkiGibson