Add more diagnostic information to better understand what's happening if TestAddAndRemoveViaTextChangeManyTimes fails

make msbuildtask not eat errors

Enable interactive commands

Use 'nameof()' in places where we were initializing a variable with the string form of its name.

Run codeformatter on our code.

fix crash.

Archive only relevant artifacts

Handle ErrorTypeSymbol as a ContainingSymbol of TypeParameterSymbol in CSharp.DocumentationCommentIDVisitor.PartVisitor.VisitTypeParameter.

Enable execute in interactive command.
Enable reset interactive from project command.

Adjust the Portable PDB validation code to account for missing VB fake locals

Fix and enable failing C# xplat syntax tests

Change our archive logic to only store the items that we actually use for investigations.

closes #8550

Code Model: Don't crash when adding element, deleting its text and then adding it again

use FormattedSymbol

Enable parallel runs of determinism job

I don't have heap so can't figure out why the situation happened. but this should at least prevent crash.

when this happens, after this fix, we might have wrong span for diagnostic, but that is not critical issue to crash VS.

In the initial move to running tests xplat, a number of tests were
disabled due to failures in xplat CI. Some of these failures are
inherent platform limitations, like tests which depend on the Windows
.NET Desktop framework, but some were simply improperly written tests.

This change re-enables and fixes the tests that were improperly or
non-portably written.

Remove unused ci perf infrastructure

  Pass through features, publicsign, and deterministic flags to the IDE

Fix DiaSymReader nuget version string calculation

Handle ErrorTypeSymbol as a ContainingSymbol of TypeParameterSymbol in CSharp.DocumentationCommentIDVisitor.PartVisitor.VisitTypeParameter.

Fixes #8523.

Add support for paths with spaces in cibuild

Replacing unreachable exception with no-op in OperationVisitor

Address performance issues in the AnalyzerManager reported at…

Stop deploying extension in BuildAndTest.proj

Ignore accessors in SetMetadataName

Change how we format on certain commit chars from completion. Fixes #7768, #8228

Fixes #8498

2ed22626 caused CIBuild's to start deploying extensions
because properties are not inherited by nested MSBuild
instances. Actually pass down DeployExtension=false.

Address performance issues in the AnalyzerManager reported at https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/2078#issuecomment-181150995:

1. Replace use of ConcurrentDictionaries with Dictionary + lock.
2. Avoid unncessary ToImmutableArray invocations.

I also fixed up a subtle race condition in CompilationAnalysisValueProvider.TryGetValue for my last PR (see https://github.com/dotnet/roslyn/pull/8375/files#r52127390).