* Deploy System.Diagnostics.DiagnosticSource to Tools folder (#14417)

* reapply test disable

* Update SymbolUse.fs

oops

Co-authored-by: NPetr Pokorny <petr@innit.cz>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.22578.1 -> To Version 8.0.0-beta.22579.2
Co-authored-by: Ndotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

* Update dependencies from https://github.com/dotnet/arcade build 20221122.5

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.22554.2 -> To Version 8.0.0-beta.22572.5

* Update dependencies from https://github.com/dotnet/arcade build 20221124.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.22554.2 -> To Version 8.0.0-beta.22574.1

* Update dependencies from https://github.com/dotnet/arcade build 20221125.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.22554.2 -> To Version 8.0.0-beta.22575.1

* Update global.json

* Remove a test

This test is failing in CI for some reason. But we don't really care about what it tests anyway - having private values in signature files, in which there's really no point

* Update dependencies from https://github.com/dotnet/arcade build 20221128.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.22554.2 -> To Version 8.0.0-beta.22578.1
Co-authored-by: Ndotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>
Co-authored-by: N0101 <0101@innit.cz>

* Make inline type hints less intrusive

* Update

* Reverting the breaking change

* IWSAM

* update

* extra tests

* Tests

* Update InlineParameterNameHints.fs

Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* Don't crash when looking up XML docs for signature with apostrophe

* Respecting manual Message property in an Exception type definition

* Fantomas applied

* Fixing failing test

Co-authored-by: NChet Husk <chusk3@gmail.com>
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* Issue investigation && repro

* Abs filepath remapping for test DSL

* itemKeyStore enabled for test workflow

* Changing behaviour for FindAllReferences and constructors

* tests for strange static init

* Making tests pass in net70

* Remove debugging code

* Reflect VS-realistic FsharpChecker settings in ProjectWorkflowBuilder (for tests)

* Fantomas applied

* PR feedback

Move to "-Svc-" pool providers.  (Eng/Common changes will be overwritten with the same values when arcade is updated)

* temp

* remove msbuild dependence from fcs

* remove temp diagnostic

* Caching

* fantomas

The problem was that we had some cached lexer state when reading new
input during FSI session. If lexer threw an error on a very first token
of new input, the cached lexer state would not get updated, so we would
assosciate old lexer state with this new token that caused an error.

Now we will invalidate that cached state at the begining of reading new
input.
Co-authored-by: NAdam Boniecki <adboniec@microsoft.com>

Merge release/dev17.3 to main

Move to "-Svc-" pool providers in release branch

Prefer nullable over other conversions

Move to "-Svc-" pool providers.  (Eng/Common changes will be overwritten with the same values when arcade is updated)

Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>

* Component tests for signature roundtrip

* No errors for non virtual members overrides

* Check for isFSharpObjModelTy as we are interested on csharp virtual methods

* More testing

* Add a feature lang preview

* More testing

* Fix PR comment