Shen Chen

Results 24 comments of Shen Chen

> > > Out of curiosity: why does it matter? Hello Brad, So currently if you run a failed Xunit test in VS, you will see this at the bottom...

Retarget this to main because 17.1 branch is not used and main is 17.1 now

@CyrusNajmabadi Could you review this PR? (It looks like Sam is OOF)

Tag @CyrusNajmabadi to revisit this : )

I feel this looks good, once all the existing comments are addressed it should be good to go

https://runfo.azurewebsites.net/search/tests/?q=started%3A~7%20definition%3Aroslyn-integration-ci%20name%3A%22roslyn.visualstudio.newintegrationtests.csharp.csharperrorlistnetcore.errorlist%20%22 ![image](https://user-images.githubusercontent.com/24360909/168904392-8b63882e-68f5-4525-9bb7-f55d123b9c54.png)

I took a closer look at this feature. From O#, it looks like, 1. It simply gets all the code action providers from [Roslyn](https://github.com/OmniSharp/omnisharp-roslyn/blob/ac7b9b8509356e39583de2b9fdf363005e6c8595/src/OmniSharp.Roslyn/Services/AbstractCodeActionProvider.cs#L33) 2. For extract interface, it creates...

It seems like related to 3 different kinds of failure The build for the first: https://runfo.azurewebsites.net/view/build/?number=142117 The build for the second: https://runfo.azurewebsites.net/view/build/?number=142078 The build for the third: https://runfo.azurewebsites.net/view/build/?number=141350