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.

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

Disable ConvertAll suggestion for .NET Core and .NET 5.0 and above #1033

Open
NN--- opened this issue Jan 25, 2023 · 3 comments
Open

Disable ConvertAll suggestion for .NET Core and .NET 5.0 and above #1033

NN--- opened this issue Jan 25, 2023 · 3 comments
Assignees

Comments

@NN---
Copy link

NN--- commented Jan 25, 2023

RCS1077 no longer relevant for modern .NET.
It should be disable when project targets .NET Core 3.1 or .NET 5.0 and above.

dotnet/runtime#38418
#736

@gpshonik
Copy link

I'm guessing the answer is going to be no, but is there any way to disable just this particular suggestion/refactoring via any configuration option?

listOfT.Select(f => M(f)).ToList()
listOfT.ConvertAll(f => M(f))

@josefpihrt josefpihrt self-assigned this Feb 25, 2024
@josefpihrt
Copy link
Collaborator

I'm thinking about do it the other way around: Do not suggest it by default and add an option to enable it:

roslynator_convert_select_to_convertall = true
roslynator_convert_linq_select_to_convertall = true

Or you can suggest better option key.

@gpshonik
Copy link

gpshonik commented Feb 26, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants