Duplicate functionality of existing LambdaRewriter.Analysis visitors
with the new Scope tree visitors and assert that the same results are
produced.

Add WorkItem attribute to SigHelp test

Disable Basic EE integration tests

Enable xunit analyzers

Fix vbc on non-English cultures

	Better preserve newlines between statements when removing an unused variable

Fix switches on creating dump files when VS instance crashes or hangs (integration tests)

Include impl assemblies in debugger packages

This required changing CurrentVersionNumber to a property.

Teach pattern-matching analyzer to detect and fix additional cases

Use the live PublishData.json file

Fix NuGet warnings

The most recent preview of 15.3 introduced a number of NuGet warnings
into our build. Most of these were due to new dependencies in the
service hub layer. This change fixes these warnings by moving to a new
version of StreamJsonRpc and adding new PackageReferences as dictated by
the warnings

Relevant links:

- https://github.com/NuGet/Home/issues/5633
- https://github.com/NuGet/Home/wiki/Warnings-and-Errors-Combined-Specs

This works around https://github.com/xunit/xunit/issues/1379

Right now this code is doing a pattern where it overrides the public
methods in derived types, and those have facts. That forces the base
type to also be public, but we can't put Facts there. This can be
cleaned up in a better way, but I want to keep this first round of
changes as mechanical as possible.

xunit.analyzers will flag public methods of types that have other
facts in them. Sometimes this catches things we missed [Fact] on, and
in other times was catching private test helpers. Make both of the
fixes.

- null check in `while` in addition to `if`
- inline type check e.g. `if ((o as T) != null)`

Merge dev15.3.x to master

Fixed 'Generate Constructor' refactoring for an abstract class.

Fixed incorrect offer to interpolate string if used string.Format with named arguments.

Fixed Make Method Asynchronous refactoring.

Fixed IntelliSense suggestion on typing dot after method group

Build locally with toolset 2.3.1

Use default tuple fields in deconstruction since fields from inferred names are marked not usable in C#7

Fixes #51051

Fixes #21050

Enable analyzers on Roslyn.sln except for two very slow ones

Add Shared attributes to code fix providers

Offer to not generate fields when generating a new constructor.

Add feature to convert from ReferenceEquals to 'is null'