ci: Fix Coveralls Bot from spamming coverage info (#1205)

Before this PR, the Coveralls Bot's been spamming the Coverage info as
PR comments. It's assumed, the bot's doing it because we have a
Coveralls upload step for every package.

This PR tries to fix this by using the built-in "parallel" upload job
feature by Coveralls itself. For now, this seems to have fixed the
problem (as seen in the PR checks themselves).
This only creates one minor drawback: There are now 6 instead of only
one check in the list.

---
Additionally, this PR changes the tests execution step (inside
`github-ci.yml`) from "_npm run unit_" back to "_npm run coverage_".
This is done because `coverage.yml` only checks inside "packages/" and
not "internal/".

Author: maxreichmann

.github/workflows/coverage.yml

@@ -1,4 +1,4 @@
-name: Coverage Reporting
+name: Coverage
 
 on:
   push:
@@ -14,8 +14,17 @@ permissions:
 
 jobs:
   coverage:
-    name: Check coverage and report to Coveralls
+    name: Check and Upload (packages/${{ matrix.package }})
     runs-on: ubuntu-24.04
+    strategy:
+      matrix:
+        package:
+          - builder
+          - cli
+          - fs
+          - logger
+          - project
+          - server
     steps:
 
     - uses: actions/checkout@v5
@@ -29,46 +38,23 @@ jobs:
       run: npm ci
 
     - name: Check coverage
+      working-directory: packages/${{ matrix.package }}
       run: npm run coverage
 
-    - name: Send report to Coveralls for package @ui5/builder
+    - name: Send parallel reports to Coveralls for all packages
       uses: coverallsapp/[email protected]
       continue-on-error: true # Do not fail the job if coverage reporting fails (e.g. service is down)
       with:
-        file: packages/builder/coverage/lcov.info
-        base-path: packages/builder
+        parallel: true
+        base-path: packages/${{ matrix.package }}
+        flag-name: ${{ matrix.package }}
 
-    - name: Send report to Coveralls for package @ui5/cli
-      uses: coverallsapp/[email protected]
-      continue-on-error: true # Do not fail the job if coverage reporting fails (e.g. service is down)
-      with:
-        file: packages/cli/coverage/lcov.info
-        base-path: packages/cli
-
-    - name: Send report to Coveralls for package @ui5/fs
-      uses: coverallsapp/[email protected]
-      continue-on-error: true # Do not fail the job if coverage reporting fails (e.g. service is down)
-      with:
-        file: packages/fs/coverage/lcov.info
-        base-path: packages/fs
-
-    - name: Send report to Coveralls for package @ui5/logger
-      uses: coverallsapp/[email protected]
-      continue-on-error: true # Do not fail the job if coverage reporting fails (e.g. service is down)
-      with:
-        file: packages/logger/coverage/lcov.info
-        base-path: packages/logger
-
-    - name: Send report to Coveralls for package @ui5/project
-      uses: coverallsapp/[email protected]
-      continue-on-error: true # Do not fail the job if coverage reporting fails (e.g. service is down)
-      with:
-        file: packages/project/coverage/lcov.info
-        base-path: packages/project
-
-    - name: Send report to Coveralls for package @ui5/server
-      uses: coverallsapp/[email protected]
-      continue-on-error: true # Do not fail the job if coverage reporting fails (e.g. service is down)
-      with:
-        file: packages/server/coverage/lcov.info
-        base-path: packages/server
+  finish:
+    needs: coverage
+    if: ${{ always() }}
+    runs-on: ubuntu-24.04
+    steps:
+      - name: Close parallel build
+        uses: coverallsapp/[email protected]
+        with:
+          parallel-finished: true

.github/workflows/github-ci.yml

@@ -37,7 +37,7 @@ jobs:
       run: npm run depcheck
 
     - name: Run unit tests
-      run: npm run unit
+      run: npm run coverage
 
     - name: Generate JSDoc documentation
       working-directory: internal/documentation