dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
14.23k stars 4.45k forks source link

Revert "Make mutable generic collection interfaces implement read-only collection interfaces (#95830)" #101644

Closed jkotas closed 2 weeks ago

jkotas commented 2 weeks ago

This change introduces ambiguous call error for calls of IList.Count in C++/CLI.

https://github.com/dotnet/wpf/pull/9055#issuecomment-2080380620

dotnet-issue-labeler[bot] commented 2 weeks ago

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.
dotnet-policy-service[bot] commented 2 weeks ago

Tagging subscribers to this area: @dotnet/area-system-collections See info in area-owners.md if you want to be subscribed.

jkotas commented 2 weeks ago

/backport to release/9.0-preview4

github-actions[bot] commented 2 weeks ago

Started backporting to release/9.0-preview4: https://github.com/dotnet/runtime/actions/runs/8860536300

TonyValenti commented 2 weeks ago

I hope this gets merged back in. I was really excited to see this commit land.

KennethHoff commented 2 weeks ago

This is very sad to see. This was one of the most exciting change due in .Net 9 for me 😭