David Federman

Add option to trim values (unless quoted) Eg: ``` 1997, Ford, E350 # Can be treated as 1997,Ford,E350 ``` Reference: https://en.wikipedia.org/wiki/Comma-separated_values#Basic_rules

Reference: https://en.wikipedia.org/wiki/Comma-separated_values#Basic_rules Examples to handle: ``` # All fields with quotes "1997","Ford","E350" # Some fields with quotes 1997,Ford,E350,"Super, luxurious truck" # Embedded quotes escaped with double-double-quotes 1997,Ford,E350,"Super, ""luxurious"" truck" #...

Now that MSBuildLocator targets netcoreapp2.0, we should be able to as well

Eg. appveyor or VSTS builds

<Sdk> element does not respect Conditions

8

### Steps to reproduce Project file ```xml ``` Command line ``` msbuild ``` ### Expected behavior Successful build ### Actual behavior Error similar to: ``` E:\tmp\test\Test.proj : error : C:\Program...

SdkResolverService throws "System.InvalidOperationException: Collection was modified" occasionally

8

I'm occasionally seeing the following exception: ``` System.InvalidOperationException: Collection was modified; enumeration operation may not execute. at System.Collections.Generic.List`1.Enumerator.MoveNextRare() at Microsoft.Build.BackEnd.SdkResolution.SdkResolverService.GetResolvers(IList`1 resolversManifests, LoggingContext loggingContext, ElementLocation sdkReferenceLocation) at Microsoft.Build.BackEnd.SdkResolution.SdkResolverService.ResolveSdkUsingResolversWithPatternsFirst(Int32 submissionId, SdkReference sdk,...

Add Microsoft.MSBuildCache

8

Add Microsoft.MSBuildCache This change adds a new pipeline which enables caching in the build. This is added as a separate pipeline for now with the eventual goal of enabling for...

To repro: ``` dotnet new mstest dotnet add package Microsoft.Build.RunVSTest --prerelease ``` Then add this test: ```cs [TestClass] public class UnitTest1 { [DataTestMethod] [DataRow("Something error:")] public void TestMethod1(string errorMessage) {...

I would expect at least the failing test cases to be listed and ideally the full error messages. Instead, "RUNVSTESTTASK : error Message:" is emitted to the console, which doesn't...

Pipelines: PublishBuildArtifacts -> PublishPipelineArtifact

7

The `PublishPipelineArtifact` task is *significantly* faster than the `PublishBuildArtifacts` task. Docs for reference: https://learn.microsoft.com/en-us/azure/devops/pipelines/artifacts/pipeline-artifacts Exact perf numbers TBD. Will edit the description once I have them.