* spacing for commas and colons

* code review

* add diagnostics

* add diagnostics

* diagnostics

* diagnostics

* diagnostics

* add diagnostics and possible fix for tp smoke tests

* fix build

* fix build

* more diagnostics

* try to fix flaky test

* fix build

* try to fix dodgy test

* Set compiler libraries to netstandard2.0"

* Clean up #ifs

* Standardize on NETSTANDARD

* more

* cleanup semicolons (ugh)

* fix build

* a little more cleanup

* little bit more cleanup

* little bit more cleanup

* little bit more documentation

* little bit more docs

* whitespace cleanup and docs

* more whitespace and code formatting

* little bit more docs

* add diagnostics

* add diagnostics

* add diagnostics

* diagnostics

* diagnostics

* diagnostics

* add diagnostics and possible fix for tp smoke tests

* fix build

* fix build

* cleanup commas

* fix indentation

* Update IlxGen.fs

* apply formatting to additional files

* improve output of script

* improve output of script

* process large expressions systematically

* add test cases

* update tests

* Make .NET Desktop fsi.exe 32-bit again and make Desktop fsiAnyCpu.exe (64-bit) the default to launch in VS #6223

* make fsc.exe 32-bit for compat

* make fsc.exe 32-bit for compat

* fix build

* fix build

* ramp up max testing

* correct optimization of linear matches

* improve diagnostics

* fix tests

* tests only on .NET Framework for now

* fix determinism

* Make .NET Desktop fsi.exe 32-bit again and make Desktop fsiAnyCpu.exe (64-bit) the default to launch in VS #6223

* make fsc.exe 32-bit for compat

* make fsc.exe 32-bit for compat

* improve diagnostics

* fix tests

* fix determinism

* Add coreclr test suite

* tests for linux

* No compiler on linux

* feedback

* fix flakey test (?)

* use runsettings

* same fix for FCS tests2

* anonymous record support for expressions in FCS

* anonymous record support for expressions in FCS

* add tests for anonymous records and fix service testing with F# interactive

* fix FCS docs

* fix build

* add diagnostics

* update other baseline

* fix final parallel case

* try again to fix test

* update test fixes

* reformat huge lines

* reformat most compiler options

* fix flakey test (?)

* use runsettings

* same fix for FCS tests2

* more long lines

* fix build

* add diagnostics

* fix final parallel case

* try again to fix test

* update test fixes

* fix help text

* fix flakey test (?)

* update exename

* use runsettings

* same fix for FCS tests2

* fix build

* add diagnostics

* add diagnostics

* fix final parallel case

* Microsoft.FSharp.Comiler --> FSharp.Compiler

* Microsoft.FSharp.Build --> FSharp.Build

* fix small mistakes

* fix build

* fix flakey test (?)

* Fix Oddities in statically resolved method constraints and method overloading

* no return type needed before proceeding to overload resolution

* fix tests

* unwind changes

* revert to be cleanup only

* revert to be cleanup only

* revert to be cleanup only

* revert to be cleanup only

* adjsut test

* test doesn't pass peverify

* test doesn't pass peverify

* apply code review

* Ensure core tests pick up fsharp.core nuget packages from artifacts build

* Feedback

* Cleanup some project stuff, and Ensure that Templates vsix contains newly built FSharp.Core nuget package.

* ignore

* Feedback

* Do not notify env sink about base constructor (fixes #3699)

* Extract function

* Do not store union case compiled name

* Update test data to report only case name range

* Always emit conditional attributes in FCS (fixes #3890)

* Add test

* Cleanup test

* Add internal compiler switch, emit conditional method calls

* Update test

* Make noConditionalErasure lowercase

* poss fix

* poss fix

* Update range.fs

* Update range.fs

* fix build

* docs and limit fix

* Update src/fsharp/range.fs
Co-Authored-By: Ndsyme <dsyme@users.noreply.github.com>

* Update range.fs

* Dive into types in type test expressions in AST visitor

* Add test

* Update VisualFSharp test

* Update fsproj

* Cleanup test

* Replace isModule with ModuleOrNamespaceKind in SymModuleOrNamespace

* Fix structure tests

* Review fixes

* Improve workflow

* back to 32bit msbuild

* move out nugets, change targets in makefile

* Detect framework

* Done

* feedback

* Fix #4946  --- Update FSC and FSI to recognize upto .net framework 472

* support really old frameworks

* Update test to satisfy net472 as well as earlier versions of dotnet

* comparer

* convert to SDK projects

* fix ci_part4

* convert to SDK projects

* Done

* Init command

* ws

* Update FSharpBuild.Directory.Build.props

* feedback

* Remove some asses from the codebase

* Remove more asses

* Fix #5531

This is a working fix for the example of #5531.
However there are still some points open before I think it's a good idea to merge this:
- Discuss the change of this proposal, is it OK to prefer overrides of methods and so ignore base implementations?
- Extend the check to the complete inheritance graph instead of a single "look-back"
- Only ignore a methinfo if the signature of both match (so respect different overloads)

@dsyme It would be great if you could have a look whether this check is allowed at this location or should appear earlier.

* Fix left points.
Signed-off-by: Nrealvictorprm <mueller.vpr@gmail.com>

* another approach to fix

* remove old approach

* Remove approach again and apply different approach.
Signed-off-by: Nrealvictorprm <mueller.vpr@gmail.com>

* Adjusting predicate to reflect correct behaviour.

* Revert to old List.merge strategy.

* Apply review and add tests
Signed-off-by: Nrealvictorprm <mueller.vpr@gmail.com>

* Try fixing weird CI failure
Signed-off-by: Nrealvictorprm <mueller.vpr@gmail.com>

* Fix test.
Signed-off-by: Nrealvictorprm <mueller.vpr@gmail.com>

* Fix and tests, unbuilt and untested

* Correct one of the tests

Got the order wrong for the output of `countBy`.

* Repair the groupBy test

groupBy test was bad: it assumed (ridiculously) that groupBy wouldn't iterate the original sequence even though it somehow knew the groups in the sequence.

* remove massive string allocations

* change NormalizedSource Source and Seq to Array computation expression

* fix line end according to the source positions

* remove pos use i

* fix some tests

* Initial FS-1065 implementation

* Undo removal of compilationrepresentation suffix and update surface area

* Whoopise, add the suffix to the impl file

* Update coreclr surface area

* Revert the FSComp changes that somehow got picked up

* newline

* Consume internal VOption module functions

* More internal voption module functions

* Optimize String.Concat calls

* Small refactor

* Added test

* Another test

* Fixing tests

* using /noca

* Fixing test

* Fixing tests again

* Minor adjustments from feedback

* Quick fix

* More tests

* Updated baselines + more tests

* Added better way to test IL directly in unit tests

* Moved StringConcat.fs tests

* Removed bad file

* Using FSharpChecker directly

* Added more tests

* Comparing lines for tests

* Better IL check output

* Small cleanup

* Fixed test