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

* Adding parser support for N-d arrays up to rank 32

* Add eager formatting of exceptions to fsi

Eval* methods of FSIEvaluationSession that are using
CompilationDiagnosticsLogger are at risk of mutating data that's
referred to in the exception before error message is printed. To prevent
that, this commit makes CompilationDiagnosticsLogger take additional
argument that can control early pre-processing of logged exceptions.

* Add a test for ValueRestriction error

* Minor fixes from code review
Co-authored-by: NAdam Boniecki <adboniec@microsoft.com>

* process references in groups in scripts and report incremental updates to ReferencedAssemblies

* update baselines

* process references in groups in scripts and report incremental updates to ReferencedAssemblies

* fix tests

* fix tests

* fix FCS build

* don't print assembly resolutions for nuget packages

* Update output.200.stdout.bsl

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

* don't start process to determine system assemblies

* code format

* fix build break

* Add failing test.

* Update range for SynAttribute.

* Use unionRanges instead of mkFileIndexRange.

* Update ranges in unit tests.

* Fix ServiceTests.

* Update failing FSharpSuite.Tests.

* Update negative tests.

* Trigger CI

* Check for ILEventDef custom attributes and make sure in this case the Obsolete attribute is raised

* Fix formatting

* Fix PR comments

Infer isreadonly from structs ("readonly struct" in C#) to avoid struct-copies just like we do with readonly methods (#13999)

Method on a readonly struct (instance or extension) will avoid a defensive copy of the struct, just like it was already done with readonly methods. This is done especially to play well with C# defined "readonly struct"

[<Obsolete>] attribute for enum cases support added.

* try to fix analysis of conjunctive patterns

* add test cases
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

More descriptive error message for discriminated union pattern matching.
This applies when unsufficient number of tupled arguments is in the pattern. The error message then displays proposals for the missing fields.
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* fix go-to-definition to symbols in large files

* fix go-to-definition to symbols in large files

* Symbols API: do not provide vanilla symbols for class constructor

* Fix

* Formatting
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>

Jagged arrays should not be rendered as  "array{n}d<_>", that was a mistake leading to conflict with multidimensional arrays.
This PR reverts this and also adds test to check regression of this feature.

* Allow parallel project analysis with an environment variable

* reformat CompilerImports.fs

* Wire in the flag from top-level, add an internal option to the FSC, add a constructor arg to the FSharpChecker.

* Fantomas formatting

* Update surface baseline

* Cleanup

* Dummy commit to trigger PR checks

* Update surface baseline after merge

* Empty commit to trigger PR check rerun

* Empty commit to trigger PR check rerun

* Restore tests/fsharp/regression/13219/test.fsx

* IL: optimize attribute cluster reading

* Set endRid when there's a single attribute

* formatter

* More fixes
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>

* Don't allow union case unnamed fields in xmldoc

* .

* fix doc

* fix

* fix

* fix

* Research of features to be removed

* Tests marked for removal

* Removing code related to AST compilation and Dynamic assembly generation

* Fantomas formatting applied

* fix suppress of debug poin

* Update test.fsx
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

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

* Fix referenceassemblygeneration for .exe files

* Tests

* Upgrade packages
* Upgrade baselines for Newtonsoft.Json hash
* Visual Studio integration: show a keyword's description in tooltip on mouseover
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

* revamp parallel checking

* simplify names

* fix diagnostics

* code formatting

* update surface area

* simplify diagnostic logging and format diagnostics eagerly when processing in parallel

* format code

* remove SplitRelatedDiagnostic

* fix build and cleanup

* further cleanup

* further cleanup

* format code

* fix flaterrors

* allow error recovery on collisions

* fix name generation to be deterministic

* fix build

* Update RecursiveSafetyAnalysis.fs

* Add flag for parallel type checking of files backed by signatures.

* Update src/Compiler/Driver/ParseAndCheckInputs.fs
Co-authored-by: NPetr Pokorny <petr@innit.cz>

* format code

* Update TypeTests.fs

* Prefix DiagnosticsLoggerProvider with I

* Remove duplicate hadSig binding.

* Add basic test for ParallelCheckingWithSignatureFiles flag.

* Add additional CI job.

* Produce binlog for ParallelCheckingWithSignatureFiles

* Update azure-pipelines.yml

* Update azure-pipelines.yml

* Update TypeTests.fs

* Update SyntaxTreeTests

* Correct code after rebase

* Correct SynModuleSigDecl in SignatureTypeTests.fs

* Format ParseAndCheckInputs.fs
Co-authored-by: NDon Syme <donsyme@fastmail.fm>
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>
Co-authored-by: NPetr Pokorny <petr@innit.cz>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>
Co-authored-by: NKevin Ransom (msft) <codecutter@hotmail.com>

Co-authored-by: NKevin Ransom (msft) <codecutter@hotmail.com>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

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

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

Co-authored-by: NEdgar <edgar.gonzalez@fundourselves.com>
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

* Custom equality on ModuleOrNamespaceKind (#13693)

* Fix nuget warning
Co-authored-by: NPetr Pokorny <petrpokorny@microsoft.com>

* Only process EmptyModuleOrNamespaces when showHeader is true.

* Update failing Cambridge tests.
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>