Replace hard-coded codecov_token with secrets/variables

Author: AArnott

.github/workflows/build.yml

@@ -12,7 +12,6 @@ on:
 env:
   DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true
   BUILDCONFIGURATION: Release
-  # codecov_token: 4dc9e7e2-6b01-4932-a180-847b52b43d35 # Get a new one from https://codecov.io/
   NUGET_PACKAGES: ${{ github.workspace }}/.nuget/packages/
 
 jobs:
@@ -64,11 +63,13 @@ jobs:
       uses: ./.github/actions/publish-artifacts
       if: cancelled() == false
     - name: 📢 Publish code coverage results to codecov.io
-      run: ./tools/publish-CodeCov.ps1 -CodeCovToken "${{ env.codecov_token }}" -PathToCodeCoverage "${{ runner.temp }}/_artifacts/coverageResults" -Name "${{ runner.os }} Coverage Results" -Flags "${{ runner.os }}"
+      run: |
+        if ('${{ secrets.CODECOV_TOKEN }}') {
+          ./tools/publish-CodeCov.ps1 -CodeCovToken '${{ secrets.CODECOV_TOKEN }}' -PathToCodeCoverage "${{ runner.temp }}/_artifacts/coverageResults" -Name "${{ runner.os }} Coverage Results" -Flags "${{ runner.os }}"
+        }
       shell: pwsh
       timeout-minutes: 3
       continue-on-error: true
-      if: env.codecov_token != ''
 
   docs:
     name: 📃 Docs

Expand-Template.ps1

@@ -7,9 +7,6 @@ Expands this template into an actual project, taking values for placeholders
 The name of the library. Should consist only of alphanumeric characters and periods.
 .PARAMETER Author
 The name to use in copyright and owner notices.
-.PARAMETER CodeCovToken
-A token obtained from codecov.io for your repo. If not specified, code coverage results will not be published to codecov.io,
-but can be added later by editing the Azure Pipelines YAML file.
 .PARAMETER CIFeed
 The `/{guid}` path to the Azure Pipelines artifact feed to push your nuget package to as part of your CI.
 .PARAMETER Squash
@@ -22,8 +19,6 @@ Param(
     [Parameter(Mandatory=$true)]
     [string]$Author,
     [Parameter()]
-    [string]$CodeCovToken,
-    [Parameter()]
     [string]$CIFeed,
     [Parameter()]
     [switch]$Squash
@@ -179,11 +174,6 @@ try {
     }
 
     $YmlReplacements = @{}
-    if ($CodeCovToken) {
-        $YmlReplacements['(codecov_token: ).*(#.*)'] = "`$1$CodeCovToken"
-    } else {
-        $YmlReplacements['(codecov_token: ).*(#.*)'] = "#`$1`$2"
-    }
     if ($CIFeed) {
         $YmlReplacements['(ci_feed: ).*(#.*)'] = "`$1$CIFeed"
     } else {

azure-pipelines.yml

@@ -26,7 +26,6 @@ parameters:
 variables:
   DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true
   BuildConfiguration: Release
-  codecov_token: 4dc9e7e2-6b01-4932-a180-847b52b43d35 # Get a new one from https://codecov.io/
   ci_feed: https://pkgs.dev.azure.com/andrewarnott/_packaging/CI/nuget/v3/index.json # Azure Artifacts feed URL
   NUGET_PACKAGES: $(Agent.TempDirectory)/.nuget/packages/
 

azure-pipelines/dotnet.yml

@@ -20,11 +20,14 @@ steps:
   displayName: 📢 Publish artifacts
   condition: succeededOrFailed()
 
-- ${{ if and(ne(variables['codecov_token'], ''), parameters.RunTests) }}:
+- ${{ if parameters.RunTests }}:
   - powershell: |
       $ArtifactStagingFolder = & "tools/Get-ArtifactsStagingDirectory.ps1"
       $CoverageResultsFolder = Join-Path $ArtifactStagingFolder "coverageResults-$(Agent.JobName)"
-      tools/publish-CodeCov.ps1 -CodeCovToken "$(codecov_token)" -PathToCodeCoverage "$CoverageResultsFolder" -Name "$(Agent.JobName) Coverage Results" -Flags "$(Agent.JobName)"
+      tools/publish-CodeCov.ps1 -CodeCovToken "$(CODECOV_TOKEN)" -PathToCodeCoverage "$CoverageResultsFolder" -Name "$(Agent.JobName) Coverage Results" -Flags "$(Agent.JobName)"
     displayName: 📢 Publish code coverage results to codecov.io
     timeoutInMinutes: 3
     continueOnError: true
+    # Set the CODECOV_TOKEN variable in your Azure Pipeline to enable code coverage reporting
+    # Get a token from https://codecov.io/
+    condition: and(succeeded(), ne(variables['CODECOV_TOKEN'], ''))