Migrate all usages of task: UseDotNet@2 to template install-net.yml

[konrad-jamrozik]

This PR contributes to:

• Migrate to global.json for .NET usage in our templates #5009

It is a follow-up to:

• Update archetype-sdk-tool-dotnet.yml BuildAndPackage job to install .NET SDK via install-net.yml / global.json #5010

It also remove unused SkipDotNetInstall parameter from the archetype-sdk-tool-dotnet.

[weshaggard]

Do we need to pass this here given it aligns with the default? We should consider removing the DotNetRuntimeVersion property completely.

cc @praveenkuttappan

[konrad-jamrozik]

@weshaggard @praveenkuttappan the variable is hardcoded to 6.x. It is used in two places: here, and for PublishTestResult@2 / testRunTitle:

- task: PublishTestResults@2
  condition: succeededOrFailed()
  inputs:
    testResultsFiles: '**/*.trx'
    testRunTitle: 'Windows DotNet $(DotNetRuntimeVersion)'
    testResultsFormat: 'VSTest'
    mergeTestResults: true

Removing the variable would probably necessitate removing the .NET version from the testRunTitle, unless you folks have some idea how to pull it into this file out from install-dotnet.yml or from global.json.

But I guess it is not a big loss if we remove the version from the test run title?

[konrad-jamrozik]

@weshaggard @praveenkuttappan OK I went ahead and removed the version variable.

[weshaggard]

Yes I would suggest to just remove the version from the title.

[konrad-jamrozik]

Overriding, as the APIView error is unrelated:

[xUnit.net 00:00:21.32]     APIViewIntegrationTests.ReviewManagerTests.AddRevisionAsync_Computes_Headings_Of_Sections_With_Diff_A [FAIL]
  Failed APIViewIntegrationTests.ReviewManagerTests.AddRevisionAsync_Computes_Headings_Of_Sections_With_Diff_A [2 s]
  Error Message:
   Microsoft.Azure.Cosmos.CosmosException : Response status code does not indicate success: NotFound (404); Substatus: 1013; ActivityId: 6ac47a94-9bf8-43d6-848a-d7e4e9e07b32; Reason: (Collection is not yet available for read. Please retry in some time.

@praveenkuttappan FYI

[konrad-jamrozik]

/check-enforcer override