Skip to content

Cherry-pick the ExplicitInterfaceImplemetation fix#54180

Merged
Cosifne merged 4 commits intodotnet:release/dev16.11from
Cosifne:dev/shech/portExplicitInterfaceImplementation
Jun 23, 2021
Merged

Cherry-pick the ExplicitInterfaceImplemetation fix#54180
Cosifne merged 4 commits intodotnet:release/dev16.11from
Cosifne:dev/shech/portExplicitInterfaceImplementation

Conversation

@Cosifne
Copy link
Copy Markdown
Member

@Cosifne Cosifne commented Jun 17, 2021

Port this fix to 16.11
#53984

@Cosifne Cosifne requested a review from a team as a code owner June 17, 2021 18:13
@ghost ghost added the Area-IDE label Jun 17, 2021
@Cosifne
Copy link
Copy Markdown
Member Author

Cosifne commented Jun 17, 2021

@jinujoseph @vatsalyaagrawal
This is just a one line fix for the inheritance margin. Without this fix inheritance margin won't be created for explict interface implementation.
So I feel it will be great to check this change to 16.11

@Cosifne
Copy link
Copy Markdown
Member Author

Cosifne commented Jun 22, 2021

OMG this is forgotten for a long time.
@davidwengier Could you give this a quick approval. it's the same as #53984

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

Why do we need to backport this to 16.11? seems like a minor issue?

@Cosifne
Copy link
Copy Markdown
Member Author

Cosifne commented Jun 22, 2021

Why do we need to backport this to 16.11? seems like a minor issue?

@CyrusNajmabadi
I believe explicit implementation is used widely, and the fix is pretty easy and low-risk.
Do we only checked in big reliability fix to 16.11 now?

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

Do we only checked in big reliability fix to 16.11 now?

I'm not sure what the 16.11 bar is at currently. Are we in any sort of QB mode now?

@Cosifne
Copy link
Copy Markdown
Member Author

Cosifne commented Jun 22, 2021

Do we only checked in big reliability fix to 16.11 now?

I'm not sure what the 16.11 bar is at currently. Are we in any sort of QB mode now?

@CyrusNajmabadi
I think 16.11 is in M1 mode. (required approval from Jinu or Vatsalya) and I have already explained this to Jinu

@Cosifne Cosifne merged commit 141ac4f into dotnet:release/dev16.11 Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants