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

Update index.md #36457

Merged
merged 2 commits into from
Jul 31, 2023
Merged

Update index.md #36457

merged 2 commits into from
Jul 31, 2023

Conversation

IEvangelist
Copy link
Member

@IEvangelist IEvangelist commented Jul 31, 2023 •
edited by github-actions bot
Loading

Fixes #36451

Internal previews

📄 File 🔗 Preview link
docs/standard/linq/index.md LINQ overview

@IEvangelist IEvangelist requested a review from a team as a code owner July 31, 2023 13:40
@dotnet-bot dotnet-bot added this to the July 2023 milestone Jul 31, 2023
@IEvangelist IEvangelist requested a review from adegeo July 31, 2023 13:40
@IEvangelist IEvangelist enabled auto-merge (squash) July 31, 2023 13:40
@ghost ghost added the okr-quality Content-quality KR: Concerns article defects (bugs), freshness, or build warnings. label Jul 31, 2023
BillWagner
BillWagner approved these changes Jul 31, 2023
View reviewed changes
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.

This looks great @IEvangelist

I made a suggestion to change the variable names so that the usage is clearer.

docs/standard/linq/index.md Outdated Show resolved Hide resolved
docs/standard/linq/index.md Outdated Show resolved Hide resolved
@IEvangelist IEvangelist merged commit 99633b7 into main Jul 31, 2023
7 checks passed
@IEvangelist IEvangelist deleted the fix-36451 branch July 31, 2023 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BillWagner BillWagner BillWagner approved these changes

@adegeo adegeo Awaiting requested review from adegeo

Assignees
No one assigned
Labels
dotnet-fundamentals/svc okr-quality Content-quality KR: Concerns article defects (bugs), freshness, or build warnings.
Projects
None yet
Milestone
July 2023
Development

Successfully merging this pull request may close these issues.

Incorrect syntax for Aggregate Linq function in example
3 participants
@IEvangelist @BillWagner @dotnet-bot