Skip to content

Use forEach() w/ an accumulator over reduce() with an ignored return value#14817

Merged
brettcannon merged 2 commits intomicrosoft:mainfrom
brettcannon:silence-sonar
Nov 25, 2020
Merged

Use forEach() w/ an accumulator over reduce() with an ignored return value#14817
brettcannon merged 2 commits intomicrosoft:mainfrom
brettcannon:silence-sonar

Conversation

@brettcannon
Copy link
Copy Markdown
Member

@brettcannon brettcannon commented Nov 24, 2020

…value

Sonar reports this as a bug since it's ignoring a return value in a situation where you don't _have_ to use a reduction/fold.
@brettcannon brettcannon added the no-changelog No news entry required label Nov 24, 2020
@brettcannon brettcannon self-assigned this Nov 24, 2020
Prettier was installed, but VS Code didn't run it due to having more than one formatter installed thanks to the built-in one.
@brettcannon brettcannon merged commit 5e938c6 into microsoft:main Nov 25, 2020
@brettcannon brettcannon deleted the silence-sonar branch November 25, 2020 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants