ci: adjust finish-coverage-report job condition for strict workflow c…

…ontrol

Signed-off-by: Marko Kungla <[email protected]>

.github/workflows/monorepo-main.yml

@@ -16,9 +16,9 @@ on:
   workflow_dispatch:
     inputs:
       strict:
-        description: 'Set to false to allow workflow to continue on errors'
+        description: 'Set to true to make the workflow fail on the first error'
         required: false
-        default: 'true'
+        default: 'false'
 
 jobs:
   lint:
@@ -51,7 +51,7 @@ jobs:
       matrix:
         module: ${{ fromJson(needs.list-modules.outputs.modules) }}
         platform: [ubuntu-latest, macos-latest, windows-latest]
-      fail-fast: ${{ github.event.inputs.strict == 'true' || github.event_name != 'workflow_dispatch' }}
+      fail-fast: ${{ github.event.inputs.strict == 'true' }}
     runs-on: ${{ matrix.platform }}
     steps:
       - uses: actions/checkout@v4
@@ -94,12 +94,12 @@ jobs:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           file: ${{ matrix.module }}/coverage.out
 
-  finish-coverage-report:
-    needs: coverage
-    if: always() && (github.event.inputs.strict == 'false' || success())
-    runs-on: ubuntu-latest
-    steps:
-      - name: Coveralls Finish
-        uses: coverallsapp/github-action@v2
-        with:
-          parallel-finished: true
+    finish-coverage-report:
+      needs: coverage
+      if: github.event.inputs.strict == 'false' || success()
+      runs-on: ubuntu-latest
+      steps:
+        - name: Coveralls Finish
+          uses: coverallsapp/github-action@v2
+          with:
+            parallel-finished: true