Skip to content

Commit 31b22c9

Browse files
Potential fix for pull request finding 'Missed opportunity to use Select'
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
1 parent 2c8f2c2 commit 31b22c9

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

src/services/Elastic.Documentation.Services/ChangelogService.cs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -495,15 +495,13 @@ AccessViolationException or
495495
private static List<string> MapLabelsToAreas(string[] labels, Dictionary<string, string> labelToAreasMapping)
496496
{
497497
var areas = new HashSet<string>();
498-
foreach (var label in labels.Where(label => labelToAreasMapping.ContainsKey(label)))
498+
var areaList = labels
499+
.Where(label => labelToAreasMapping.ContainsKey(label))
500+
.SelectMany(label => labelToAreasMapping[label]
501+
.Split(',', StringSplitOptions.RemoveEmptyEntries | StringSplitOptions.TrimEntries));
502+
foreach (var area in areaList)
499503
{
500-
var mappedAreas = labelToAreasMapping[label];
501-
// Support comma-separated areas
502-
var areaList = mappedAreas.Split(',', StringSplitOptions.RemoveEmptyEntries | StringSplitOptions.TrimEntries);
503-
foreach (var area in areaList)
504-
{
505-
_ = areas.Add(area);
506-
}
504+
_ = areas.Add(area);
507505
}
508506
return areas.ToList();
509507
}

0 commit comments

Comments
 (0)