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

[release/9.0-preview4] Revert "Make mutable generic collection interfaces implement read-only collection interfaces (#95830)" #101645

Closed github-actions[bot] closed 2 weeks ago

github-actions[bot] commented 2 weeks ago

Backport of #101644 to release/9.0-preview4

/cc @jkotas

Customer Impact

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

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

Regression

https://github.com/dotnet/runtime/pull/95830

Testing

[How was the fix verified? How was the issue missed previously? What tests were added?]

Risk

[High/Medium/Low. Justify the indication by mentioning how risks were measured and addressed.]

IMPORTANT: If this backport is for a servicing release, please verify that:

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.