fix: Normalize the option to post coverage as PR comment

insightsengineering · Feb 13, 2024 · fe49be2 · fe49be2
1 parent 01672bb
commit fe49be2
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/.github/workflows/test-coverage.yaml b/.github/workflows/test-coverage.yaml
@@ -158,10 +158,16 @@ jobs:
             echo "ref-name=${{ steps.branch-name.outputs.current_branch }}" >> $GITHUB_OUTPUT
           fi
           # Check if pkgdown multiversion docs are used at all.
+          echo "multiversion-docs=false" >> $GITHUB_OUTPUT
           if [ $(grep -rl '<!-- start dropdown for versions -->' gh-pages | wc -l) -gt 0 ]; then
             echo "multiversion-docs=true" >> $GITHUB_OUTPUT
-          else
-            echo "multiversion-docs=false" >> $GITHUB_OUTPUT
+          fi
+          # Normalize the 'publish-coverage-report' option
+          echo "publish-coverage-report=${{ inputs.publish-coverage-report }}" >> $GITHUB_OUTPUT
+          if [ "${{ inputs.publish-coverage-report }}" == "" ]
+          then {
+            echo "publish-coverage-report=true" >> $GITHUB_OUTPUT
+          }
           fi
         shell: bash
 
@@ -271,7 +277,7 @@ jobs:
           path: ${{ github.event.repository.name }}/${{ inputs.package-subdirectory }}/coverage.xml
           threshold: 80
           fail: false
-          publish: ${{ inputs.publish-coverage-report }}
+          publish: ${{ steps.current-branch-or-tag.outputs.publish-coverage-report }}
           diff: true
         continue-on-error: true