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

Incorrect syntax for Aggregate Linq function in example #36451

Closed
underager opened this issue Jul 30, 2023 · 0 comments · Fixed by #36457
Closed

Incorrect syntax for Aggregate Linq function in example #36451

underager opened this issue Jul 30, 2023 · 0 comments · Fixed by #36457
Assignees
Labels
doc-bug Problem with the content; needs to be fixed [org][type][category] dotnet-fundamentals/svc okr-quality Content-quality KR: Concerns article defects (bugs), freshness, or build warnings. Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Identifies that an issue has been imported into Quest.

Comments

@underager
Copy link

underager commented Jul 30, 2023

In the link mentioned in the Content. Go to section Essential LINQ.
In that the code example with the header The bread and butter - Where, Select, and Aggregate

The last section of example should include the type of 'strings'.
If it is as follows
string[] strings = {"abc", "xyzw"};

Then do the following change in example text.
Instead of having it like this

int sumOfStrings = strings.Aggregate(seed, (s1, s2) => s1.Length + s2.Length);

It should be as follows

int sumOfStrings = strings.Aggregate(seed, (s1, s2) => s1 + s2.Length);


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.


Associated WorkItem - 143922

@dotnet-bot dotnet-bot added the ⌚ Not Triaged Not triaged label Jul 30, 2023
@IEvangelist IEvangelist self-assigned this Jul 31, 2023
@IEvangelist IEvangelist added doc-bug Problem with the content; needs to be fixed [org][type][category] okr-quality Content-quality KR: Concerns article defects (bugs), freshness, or build warnings. 🗺️ reQUEST Triggers an issue to be imported into Quest. labels Jul 31, 2023
@dotnet-bot dotnet-bot removed the ⌚ Not Triaged Not triaged label Jul 31, 2023
@IEvangelist IEvangelist added Pri1 High priority, do before Pri2 and Pri3 and removed Pri2 labels Jul 31, 2023
IEvangelist added a commit that referenced this issue Jul 31, 2023
@ghost ghost added the in-pr This issue will be closed (fixed) by an active pull request. label Jul 31, 2023
@IEvangelist IEvangelist moved this from 🔖 Ready to 👀 In review in dotnet/docs July 2023 sprint Jul 31, 2023
@IEvangelist IEvangelist changed the title Error in example Incorrect syntax for Aggregate Linq function in example Jul 31, 2023
IEvangelist added a commit that referenced this issue Jul 31, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in dotnet/docs July 2023 sprint Jul 31, 2023
@ghost ghost removed the in-pr This issue will be closed (fixed) by an active pull request. label Jul 31, 2023
@github-actions github-actions bot added 📌 seQUESTered Identifies that an issue has been imported into Quest. and removed 🗺️ reQUEST Triggers an issue to be imported into Quest. labels Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-bug Problem with the content; needs to be fixed [org][type][category] dotnet-fundamentals/svc okr-quality Content-quality KR: Concerns article defects (bugs), freshness, or build warnings. Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Identifies that an issue has been imported into Quest.
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants