Skip to content

Infer better variable names for a foreach expression.#42411

Merged
CyrusNajmabadi merged 10 commits intodotnet:masterfrom
CyrusNajmabadi:pluralizeVarNames
Mar 19, 2020
Merged

Infer better variable names for a foreach expression.#42411
CyrusNajmabadi merged 10 commits intodotnet:masterfrom
CyrusNajmabadi:pluralizeVarNames

Conversation

@CyrusNajmabadi
Copy link
Contributor

Fixes #56

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner March 14, 2020 03:08
@CyrusNajmabadi CyrusNajmabadi requested a review from mavasani March 14, 2020 03:08
using Microsoft.CodeAnalysis.Utilities;
using Roslyn.Utilities;
using Humanizer;
using Microsoft.CodeAnalysis;
Copy link
Member

Choose a reason for hiding this comment

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

nit - sorting

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

@CyrusNajmabadi CyrusNajmabadi merged commit 179d8d6 into dotnet:master Mar 19, 2020
@ghost ghost added this to the Next milestone Mar 19, 2020
@CyrusNajmabadi CyrusNajmabadi deleted the pluralizeVarNames branch March 19, 2020 06:45
@sharwell sharwell modified the milestones: Next, temp, 16.6.P3 Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introduce Local should look for naming patterns in ForEachStatementSyntax.Expression

5 participants