Rikki Gibson

@jcouv for second review

> The doc still says that we treat `#line` and `InterceptsLocation` paths differently which I don't understand. They should be the same for the new version of the attribute. Are...

Windows tests runs are timing out here. Also seeing a warning in the test project: `/home/vsts/work/1/s/tests/Formatters/OrganizeImportsFormatterTests.cs(3,1): warning IDE0005: Using directive is unnecessary. [/home/vsts/work/1/s/tests/dotnet-format.UnitTests.csproj]`

This is still happening. I wonder if you would accept a fix for this @bradwilson? Should it work by removing the escaping and fixing up consumers that expected the escaping,...

Posting just in case it helps anyone. I have created a bookmarklet which reverses the undesirable HTML escaping. You can open your HTML test results in your browser and just...

(sorry, I misunderstood the description in my original comment.) It does feel like there's an excessive amount of fixed-size overhead here.

A trace from a sampling profiler of some kind could be helpful. My guess though is that a lot of this overhead is in just spinning up the runtime, loading...

Using the `--folder` option and formatting only whitespace should be much faster than formatting based on a project or sln file. I'm certainly not happy with the level of performance...

The Roslyn repo has a CI job which runs a build with analyzers, including [IDE0055](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0055), at error severity. This prevents people from checking in code with bad formatting.

I haven't been experiencing any email firehose problems in recent sprints so will close out.