[Bug fix]Fix instrumentation serialization bug

[MarcoRossignoli]

Fixes #452

• I changed serializer, now we use DataContractSerializer that doesn't need converters and follow guidelines
  https://docs.microsoft.com/en-us/visualstudio/code-quality/ca2300-do-not-use-insecure-deserializer-binaryformatter?view=vs-2019
  https://docs.microsoft.com/en-us/visualstudio/code-quality/ca2301-do-not-call-binaryformatter-deserialize-without-first-setting-binaryformatter-binder?view=vs-2019

• Downgraded Newtosoft.Json ref in core dll because we risk to have issue with old SDK.

• Updated debug doc with new /p:CoverletToolsPath usage thank's to @AArnott update, very useful.

Tested with @AdmiringWorm repo

D:\git\Cake.Codecov\Source\Cake.Codecov.Tests (develop -> origin)
λ dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   2.1.603
 Commit:    ae71c68742
...
D:\git\Cake.Codecov\Source\Cake.Codecov.Tests (develop -> origin)
λ dotnet test /p:CollectCoverage=true /p:Exclude="[xunit.*]*" /p:CoverletToolsPath=D:\git\coverlet\src\coverlet.msbuild.tasks\bin\Debug\netstandard2.0\
Test run for D:\git\Cake.Codecov\Source\Cake.Codecov.Tests\bin\Debug\netcoreapp2.0\Cake.Codecov.Tests.dll(.NETCoreApp,Version=v2.0)
Microsoft (R) Test Execution Command Line Tool Version 16.0.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
[xUnit.net 00:00:00.76]     Cake.Codecov.Tests.CodecovRunnerTests.Should_Use_Codecov_Runner_From_Tool_Path_If_Provided_On_Unix [SKIP]
Skipped  Cake.Codecov.Tests.CodecovRunnerTests.Should_Use_Codecov_Runner_From_Tool_Path_If_Provided_On_Unix

Total tests: 33. Passed: 32. Failed: 0. Skipped: 1.
Test Run Successful.
Test execution time: 1.8148 Seconds

Calculating coverage result...
  Generating report 'D:\git\Cake.Codecov\Source\Cake.Codecov.Tests\coverage.json'

+--------------+------+--------+--------+
| Module       | Line | Branch | Method |
+--------------+------+--------+--------+
| Cake.Codecov | 100% | 100%   | 100%   |
+--------------+------+--------+--------+

+---------+------+--------+--------+
|         | Line | Branch | Method |
+---------+------+--------+--------+
| Total   | 100% | 100%   | 100%   |
+---------+------+--------+--------+
| Average | 100% | 100%   | 100%   |
+---------+------+--------+--------+

Test run for D:\git\Cake.Codecov\Source\Cake.Codecov.Tests\bin\Debug\net46\Cake.Codecov.Tests.dll(.NETFramework,Version=v4.6)
Microsoft (R) Test Execution Command Line Tool Version 16.0.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
[xUnit.net 00:00:00.76]     Cake.Codecov.Tests.CodecovRunnerTests.Should_Use_Codecov_Runner_From_Tool_Path_If_Provided_On_Unix [SKIP]
Skipped  Cake.Codecov.Tests.CodecovRunnerTests.Should_Use_Codecov_Runner_From_Tool_Path_If_Provided_On_Unix

Total tests: 33. Passed: 32. Failed: 0. Skipped: 1.
Test Run Successful.
Test execution time: 2.0821 Seconds

Calculating coverage result...
  Generating report 'D:\git\Cake.Codecov\Source\Cake.Codecov.Tests\coverage.json'

+--------------+------+--------+--------+
| Module       | Line | Branch | Method |
+--------------+------+--------+--------+
| Cake.Codecov | 100% | 100%   | 100%   |
+--------------+------+--------+--------+

+---------+------+--------+--------+
|         | Line | Branch | Method |
+---------+------+--------+--------+
| Total   | 100% | 100%   | 100%   |
+---------+------+--------+--------+
| Average | 100% | 100%   | 100%   |
+---------+------+--------+--------+

Tested also with "old" code and it break with sdk 2.1.603 as expected.

cc: @tonerdo

[JSkimming]

This bug fix has worked, as demonstrated by this PR JSkimming/Castle.Core.AsyncInterceptor#72 where I've taken the nightly build.

[JSkimming]

@MarcoRossignoli are you looking to get this out in a NuGet.org release? It'll help improve the score with dependabot 😉

[MarcoRossignoli]

dependabot badge is cute!I think we should add to readme.

@JSkimming I think so we need to release also for collectors side #461 (comment)
@tonerdo takes care of nuget release, just a little patience 👍 !

[JSkimming]

Understood 😃

[MarcoRossignoli]

@JSkimming you can use nightly for the moment