Remove build.proj in favor of regular dotnet commands

Simple removal of the build.proj file allows `dotnet` commands to just work without having to specify the sln file explicitly, which is really nice.

The build.proj was building all projects 3 times (build, test, and pack all rebuilt everything and repeated package restore, etc.), so it was slower than necessary anyway.

In its place:
1. folks can just use `dotnet build` or whatever command suits them.
1. Azure Pipelines simply executes dotnet commands, which I break up into 4 steps so the pipeline can show timings for each step, etc.

Because we use the Azure Pipelines DotNetCoreCLI task for running tests, the test results are automatically collected and reported as a searchable database.
Test results can be analyzed over time to see how they perform, which ones are unstable, etc.

Author: AArnott

CONTRIBUTING.md

@@ -2,33 +2,22 @@
 
 Contributions are highly welcome, however, except for very small changes, kindly file an issue and let's have a discussion before you open a pull request.
 
-## Building The Project
+## Building the Project
 
 Clone this repo:
 
-```bash
-git clone https://github.com/tonerdo/coverlet
-```
+    git clone https://github.com/tonerdo/coverlet.git
+    cd coverlet
 
-Change directory to repo root:
+Building, testing, and packing use all the standard dotnet commands:
 
-```bash
-cd coverlet
-```
+    dotnet build
+    dotnet test
+    dotnet pack
 
-Execute build script:
+To see code coverage results while testing, run with a few extra parameters:
 
-```bash
-dotnet msbuild build.proj
-```
-
-This will result in the following:
-
-* Restore all NuGet packages required for building
-* Build and publish all projects. Final binaries are placed into `<repo_root>\build\<Configuration>`
-* Build and run tests
-
-These steps must be followed before you attempt to open the solution in an IDE (e.g. Visual Studio, Rider) for all projects to be loaded successfully.
+    dotnet test /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:Include=[coverlet.*]*
 
 ## Performance testing
 
@@ -48,8 +37,8 @@ For more realistic testing it is recommended to try out any changes to the hit c
 
 Change to the directory of the library and run the msbuild code coverage command:
 
-    dotnet msbuild /t:BuildAndTest /p:Coverage=true
-    
+    dotnet test /p:Coverage=true
+
 To run with a development version of coverlet call `dotnet run` instead of the installed coverlet version, e.g.:
 
-    dotnet msbuild /t:BuildAndTest /p:Coverage=true /p:CoverageExecutablePath="dotnet run -p C:\...\coverlet\src\coverlet.console\coverlet.console.csproj"
+    dotnet test /p:Coverage=true /p:CoverageExecutablePath="dotnet run -p C:\...\coverlet\src\coverlet.console\coverlet.console.csproj"

build.proj

@@ -4,5 +4,18 @@ steps:
     version: 2.2.300
   displayName: Install .NET Core SDK
 
-- script: dotnet msbuild build.proj /p:Configuration=$(buildConfiguration)
-  displayName: 'Run Build'
+- script: dotnet restore
+  displayName: Restore packages
+
+- script: dotnet build -c $(BuildConfiguration) --no-restore
+  displayName: Build
+
+- task: DotNetCoreCLI@2
+  displayName: Test
+  inputs:
+    command: test
+    arguments: -c $(BuildConfiguration) --no-build /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:Include=[coverlet.*]*
+    testRunTitle: $(Agent.JobName)
+
+- script: dotnet pack -c $(BuildConfiguration)
+  displayName: Pack

build.yml

@@ -1,12 +1,12 @@
 pool:
   vmImage: 'windows-2019'
 steps:
-- task: DotNetCoreInstaller@0
+- task: UseDotNet@2
   inputs:
-    version: '2.2.300'
+    version: 2.2.300
 - powershell:
     .\eng\nightly.ps1 -apiKey $env:APIKEY -source $env:SOURCE
-  ignoreLASTEXITCODE: "true"
+  ignoreLASTEXITCODE: true
   env:
     APIKEY: $(apikey)
-    SOURCE: $(source)
+    SOURCE: $(source)

eng/azure-pipelines-nightly.yml

@@ -13,7 +13,7 @@ Write-Host -ForegroundColor Blue Publish with .NET CLI
 & dotnet --info
 
 Write-Host -ForegroundColor Green Create Packages
-& dotnet msbuild "$PSScriptRoot\..\build.proj" /t:CreateNuGetPackage /p:Configuration=Release /p:PublicRelease=false # amend build.proj path if changes
+& dotnet pack -c Release
 
 Write-Host -ForegroundColor Green Upload Packages
-& dotnet nuget push "$PSScriptRoot\..\build\Release\*.nupkg" -k $apiKey -s $source
+& dotnet nuget push "$PSScriptRoot\..\build\Release\*.nupkg" -k $apiKey -s $source