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

Implement LINQ optimization: OrderBy with Identity => Order #1520

Closed
wants to merge 0 commits into from
Closed

Implement LINQ optimization: OrderBy with Identity => Order #1520

wants to merge 0 commits into from

Conversation

BenjaminBrienen
Copy link
Contributor

Solves #1492

TODO:

  • Unit test fails because the unit test is run with a version of .NET Core where System.Linq does not yet implement IEnumerable<T>.Order().

@BenjaminBrienen
Copy link
Contributor Author

@josefpihrt Would you be able to point me in the right direction? Maybe an example of an analyzer/code fix that only activates when using at least a certain version of .NET?

@BenjaminBrienen
Copy link
Contributor Author

@dotnet-policy-service agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@BenjaminBrienen