Merge remote-tracking branch 'upstream/release/dev16.3-preview1' into features/lspSupport_merge_lsp_only

Add only LSP server projects without liveshare dependencies.

make sure we only create OB when VS is running in IDE mode. all comma…

Merge master to release/dev16.3-preview1

Related to #35619.

Merge master to release/dev16.3-preview1

Fix GetHashCode on constructed types with annotated type arguments

set "DeferUntilIntellisenseIsReady"=dword:00000000 for csharp and VB …

Merge master to release/dev16.3-preview1

Conflicts:
	src/Compilers/CSharp/Test/Semantic/Semantics/NullableReferenceTypesTests.cs

Handle lazy loading of analyzer command handlers

Add workarounds to IDE analyzers for missing IOperation/CFG support for using declarations

set "DeferUntilIntellisenseIsReady"=dword:00000000 for csharp and VB editor so that we opt out from VS waiting for solution load to complete

make sure we only create OB when VS is running in IDE mode. all command line mode (and all its variations) we don't need to create OB

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19324.24

Pass feature flags to the remote workspace

Fix ConvertLocalFunctionToMethod not available for selection + initial work on common helpers.

Nullable Analysis Pay-for-play

Fix typo in task namespace in docs

Merge master to release/dev16.3-preview1

* Correctly report nullability of tuples in the semantic model:
- Store the original tuple literal rather than just its type
- Visit the original literal without diagnostics only in the nullability walker to understand its type
- Update the generator to allow force visiting fields in the nullable re-writer
- Update type checker to allow checking of converted types too
- Update various tuple tests to be correct
- Add extra tests

This implements IExperimentationService in the remote workspace; any
request for feature flags is forwarded to the main Visual Studio process
and the feature flag is queried there.
Co-authored-by: NJoey Robichaud <jorobich@microsoft.com>
Co-authored-by: NHeeJae Chang <hechang@microsoft.com>

Couple of AD0001 fixes for unused parameter analyzer

Unwrap nullability-wrapped symbols in object initializer completion 

Fixes #36304

Fixes #33464

Workaround for #32100 for Dev16.2

Fixes #34830

Fixes #36715

This slipped through because we didn't have any tests at all for
the case where we are using the type-inferrer to infer the case like:

    a = new A { B = { } }

where the inner one doesn't have a new B before it. The scenario wasn't
new but just happened to be the precise case that was broken.

Fixes https://github.com/dotnet/roslyn/issues/36702

Fixes #30324