Revert "upload entire coverage report directory, and remove coverage threshold check pending cequence-io#24"

This reverts commit 6420245.

Author: phelps-sg

.github/workflows/continuous-integration.yml

@@ -47,11 +47,17 @@ jobs:
       - name: Build & Test
         run: sbt ++${{ matrix.scala }} clean testWithCoverage
 
-      - name: Upload coverage report (all)
+      - name: Upload coverage report (Cobertura)
         uses: actions/[email protected]
         with:
-          name: coverage-report-${{ matrix.scala }}
-          path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/coverage-report
+          name: cobertura.xml
+          path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/coverage-report/cobertura.xml
+
+      - name: Upload coverage report (HTML)
+        uses: actions/[email protected]
+        with:
+          name: scoverage-report-html
+          path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/scoverage-report/
 
   optional-build:
     name: Build (Optional)
@@ -86,28 +92,28 @@ jobs:
       - name: Perform Build / Test
         run: sbt ++${{ matrix.scala }} clean compile test
 
-#  coverage:
-#    name: Coverage Report
-#    if: ${{ github.event.pull_request }}
-#    needs: [ build ]
-#    runs-on: ubuntu-latest
-#    steps:
-#      - uses: actions/download-artifact@v3
-#        with:
-#          name: cobertura.xml
-#
-#      - name: Analyzing coverage report
-#        uses: 5monkeys/cobertura-action@master
-#        with:
-#          path: cobertura.xml
-#          only_changed_files: true
-#          fail_below_threshold: true
-#          show_missing: true
-#          show_line: true
-#          show_branch: true
-#          show_class_names: true
-#          link_missing_lines: true
-#          minimum_coverage: 75
+  coverage:
+    name: Coverage Report
+    if: ${{ github.event.pull_request }}
+    needs: [ build ]
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/download-artifact@v3
+        with:
+          name: cobertura.xml
+
+      - name: Analyzing coverage report
+        uses: 5monkeys/cobertura-action@master
+        with:
+          path: cobertura.xml
+          only_changed_files: true
+          fail_below_threshold: true
+          show_missing: true
+          show_line: true
+          show_branch: true
+          show_class_names: true
+          link_missing_lines: true
+          minimum_coverage: 75
 
   ready-to-merge:
     name: Ready to Merge