* TailCalls test migration

* fix the diff in IL

* fix for netcore

* Recover ValNoMutable and PropertyCannotBeSet errors

* Recover FieldNotMutable error

* Refactor tests

* Refactor tests

* Update baseline

* Fix member declaration ranges

* Fix member declaration ranges

* Update baseline

* Update baseline

* Optimize Seq.Last like Seq.Length

Add type check to Seq.Last and Seq.TryLast to avoid full iteration if not necessary. Like Seq.Length does it at
https://github.com/dotnet/fsharp/blob/c18e1780b3f3f345364cb1ad8e510ea9f4590d3a/src/fsharp/FSharp.Core/seq.fs#L709

* Add test for Optimized Seq.Last and Seq.TryLast

* style update

* style update 2

* update comments and fix build error

Github Build error was:
 Check failure on line 132 in tests\FSharp.Core.UnitTests\FSharp.Core\Microsoft.FSharp.Collections\SeqModule2.fs

@azure-pipelines
azure-pipelines
/ fsharp-ci (Build Windows vs_release)

tests\FSharp.Core.UnitTests\FSharp.Core\Microsoft.FSharp.Collections\SeqModule2.fs#L132
tests\FSharp.Core.UnitTests\FSharp.Core\Microsoft.FSharp.Collections\SeqModule2.fs(132,58): error FS0064: This construct causes code to be less generic than indicated by the type annotations. The type variable 'a has been constrained to be type 'unit'.

* include List.last case

* include list in tests

* ensure same exception is raised on empty list

* typo

* Added implementation of List.last since it is not available at this point due to compilation error

The recursive function could also be defined inside but i guess then it would be reallocated on every call to Seq.Last (in case of list match).
An alternative would be to add .Last member on list type (like .Length member)

* inline List.last

reverting my previous attempt  https://github.com/dotnet/fsharp/pull/7765/commits/b329e23c543b41b36143fafc77187acc58585471
tail rec functions should be inlined

* typo in tests(build failed)

* typo2 in test (Build failed)

* move implemnetation to internal module

as suggested by dsyme

* renamed internal tryLast to tryLastV

* Fix some issues in ValueNone implementation

* More debug display

* Some for options in debug display

* Apply suggestions from code review
Co-Authored-By: NEugene Auduchinok <eugene.auduchinok@gmail.com>

* Probably better stringing

* Add them baselines

* Add sprintfn tests

* Update OptionModule.fs

* Ref assemblies for non sdk apps

* Use NumberStyles.AllowDecimalPoint with Parse

* Correctly this time

* Improved Assembly search

[master] Update dependencies from dotnet/arcade

Cherry pick fix for eval not returning a value for multi-line submissions.

* return the last-computed value when evaluating script code

* manually track last produced value

* set last value on all branches

A recent update to the `ubuntu-16.04` VM image in ADO causes mono/fake to crash and this is blocking all work.  The MacOS version works and follows an identical code path.

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19502.11

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19501.4

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19480.3

* Consider '_' for members with access modifiers

* Add tests

* Revert

* Add tests with modifiers

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19477.2

Only the 16.3 build uses this file and that branch still has a copy.

replace deprecated nuspec `iconUrl` element with `icon`

Merge remote-tracking branch 'upstream/darc-master-ccc2f9a8-8c49-40d2-a189-a6dc389f609c' into package-icon

* Change cancellation test to not depend on how AwaitTask works

This test is to check that when an Async is started as a Task (via
StartAsTask) that when cancelled the Task isn't immediately marked as
cancelled but instead waits for the underlying Async to finish (normally
via cancellation)

* Add test that AwaitTask ignore async cancellation

- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19476.6

add build telemetry for Linux/MacOS builds