Skip to content

Conversation

@CyrusNajmabadi
Copy link
Member

Extracted from #41510 to make that PR easier to review.

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner February 8, 2020 11:04
@CyrusNajmabadi
Copy link
Member Author

Tagging @mavasani

@mavasani mavasani self-assigned this Feb 9, 2020
@jinujoseph jinujoseph added Area-IDE Community The pull request was submitted by a contributor who is not a Microsoft employee. labels Feb 10, 2020
@mavasani
Copy link
Contributor

Thanks @CyrusNajmabadi.

@dotnet/roslyn-ide Anyone else wants to take a look? This is just cleaning up code to remove some internal API dependencies so it can eventually be pulled into the shared layer to be consumed by CodeStyle layer.

Copy link
Contributor

@mavasani mavasani left a comment

Choose a reason for hiding this comment

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

Awesome, thanks @CyrusNajmabadi and @sharwell

@CyrusNajmabadi
Copy link
Member Author

This has passed testin. Feel free to merge @sharwell @mavasani Thanks!

@mavasani mavasani merged commit 1a4de48 into dotnet:master Feb 10, 2020
@CyrusNajmabadi CyrusNajmabadi deleted the fixAllCleaup branch February 10, 2020 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-IDE Community The pull request was submitted by a contributor who is not a Microsoft employee.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants