Skip to content

Conversation

@fcdeveloper00
Copy link
Contributor

Hi, First of all, thanks a lot for your efforts and your rich contents. In the first lines of 'GroupByPropertyMethod' method's body, which are commented lines, the query variable name should be changed to GroupByYearQuery , and also the type of it should be modified to IEnumerable<IGrouping<GradeLevel,DataClass.Student>>.

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

Hi, First of all, thanks a lot for your efforts and your rich contents.
In the first lines of 'GroupByPropertyMethod' method, which is  commented lines, the query variable name should be changed to GroupByYearQuery , and also the type of it to
IEnumerable<IGrouping<GradeLevel,DataClass.Student>>.
@fcdeveloper00 fcdeveloper00 requested review from a team and BillWagner as code owners June 15, 2025 08:36
@dotnetrepoman dotnetrepoman bot added this to the June 2025 milestone Jun 15, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-csharp/svc csharp-linq/subsvc community-contribution Indicates PR is created by someone from the .NET community. labels Jun 15, 2025
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @fcdeveloper00

This LGTM, and I'll :shipit: now.

@BillWagner BillWagner enabled auto-merge (squash) June 16, 2025 15:36
@BillWagner BillWagner merged commit 9502663 into dotnet:main Jun 16, 2025
12 checks passed
@fcdeveloper00 fcdeveloper00 deleted the patch-4 branch August 7, 2025 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. csharp-linq/subsvc dotnet-csharp/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants