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

EmptyDefaultValueProvider should support IReadOnlyCollection #1291

Open
coenvk opened this issue Sep 30, 2022 · 3 comments
Open

EmptyDefaultValueProvider should support IReadOnlyCollection #1291

coenvk opened this issue Sep 30, 2022 · 3 comments
Labels
duplicate enhancement

Comments

@coenvk
Copy link

coenvk commented Sep 30, 2022 •
edited by devlooped-bot
Loading

The EmptyDefaultValueProvider should support IReadOnlyCollection

Back this issue
Back this issue
@stakx stakx changed the title Moq fails when trying to return an IReadOnlyCollection EmptyDefaultValueProvider should support IReadOnlyCollection Sep 30, 2022
@fndejan
Copy link

fndejan commented Oct 4, 2022

Hi @stakx, is anyone working on this one? I would like to take it.

@stakx
Copy link
Contributor

stakx commented Oct 4, 2022 •
edited
Loading

Hi @fndejan, no, noone is working on this right now. This (or something very similar) has been requested before, so please be aware of #173 (comment). I can definitely see the sense behind supporting newer common collection types, and I'm a little more willing than back then to make reasonable breaking changes... but EmptyDefaultValueProvider is such a core part of Moq that it has the potential to break lots of existing code. Therefore I'm still undecided whether or not such a change should be merged. I'd like to think this through and try to weigh possible consequences more carefully.

@stakx stakx added the duplicate label Oct 4, 2022
@github-actions GitHub Actions
Copy link

Due to lack of recent activity, this issue has been labeled as 'stale'.
It will be closed if no further activity occurs within 30 more days.
Any new comment will remove the label.

@github-actions github-actions bot added the stale label Aug 24, 2024
@github-actions github-actions bot removed the stale label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@stakx @coenvk @fndejan