RCS1077 for OrderBy(x => x)

[bricelam]

RCS1077 should recommend replacing .OrderBy(x => x) with .Order().

[BenjaminBrienen]

This seems like a beginner-friendly task and I'm interested in contributing. I'll see about doing this this weekend if that's ok.

[josefpihrt]

@BenjaminBrienen You are more than welcome to do so!

Please see this file where you can a reference to the analyzer and cod fix provider:

roslynator/src/Tests/Analyzers.Tests/RCS1077OptimizeLinqMethodCallTests.cs

Line 11 in 6a0519c

public class RCS1077OptimizeLinqMethodCallTests : AbstractCSharpDiagnosticVerifier<InvocationExpressionAnalyzer, OptimizeLinqMethodCallCodeFixProvider>

Also, following file can be useful even though you are not adding new analyzer:

https://josefpihrt.github.io/docs/roslynator/how-to-add-new-analyzer

[BenjaminBrienen]

I love it when I can just git clone a repository and all the tests pass! (except the test I just added)

[BenjaminBrienen]

Analysis is working, working on code fix

[BenjaminBrienen]

Apparently Order() only exists new newer versions of .NET. The code fix works though!

https://learn.microsoft.com/en-us/dotnet/api/system.linq.enumerable.order?view=net-8.0&viewFallbackFrom=net-6.0

[BenjaminBrienen]

I found something that I could use
#1157 (comment)