Fix coverage merging (#12)

* fix coverage

* remove platform arg

* fix

Author: zeriyoshi

.github/octocov.yml

@@ -4,8 +4,6 @@ coverage:
   acceptable: current >= 60%
 testExecutionTime:
   if: false
-comment:
-  if: is_pull_request
 diff:
   datastores:
     - artifact://${GITHUB_REPOSITORY}

.github/workflows/coverage.yaml

@@ -13,14 +13,13 @@ on:
       - reopened
 jobs:
   Linux:
-    runs-on: ubuntu-latest
     strategy:
       matrix:
+        runs-on: ['ubuntu-24.04', 'ubuntu-24.04-arm']
         version: ['8.4']
         type: ['cli', 'zts']
         distro: ['bookworm']
-    outputs:
-      matrix: ${{ toJson(matrix) }}
+    runs-on: ${{ matrix.runs-on }}
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -30,14 +29,14 @@ jobs:
         uses: docker/setup-buildx-action@v3
       - name: Build container
         run: |
-          docker compose build --pull --no-cache --build-arg PLATFORM="linux/amd64" --build-arg IMAGE="php" --build-arg TAG="${{ matrix.version }}-${{ matrix.type }}-${{ matrix.distro }}"
+          docker compose build --pull --no-cache --build-arg IMAGE="php" --build-arg TAG="${{ matrix.version }}-${{ matrix.type }}-${{ matrix.distro }}"
       - name: Test with gcov
         run: |
           docker compose run -v "$(pwd)/ext:/ext" --rm shell pskel coverage
       - name: Upload coverage to artifact
         uses: actions/upload-artifact@v4
         with:
-          name: coverage-${{ matrix.version }}-${{ matrix.type }}-${{ matrix.distro }}
+          name: coverage-${{ matrix.runs-on }}-${{ matrix.version }}-${{ matrix.type }}-${{ matrix.distro }}
           path: ${{ github.workspace }}/ext/lcov.info
   Coverage:
     needs: [Linux]
@@ -47,17 +46,35 @@ jobs:
         uses: actions/checkout@v4
       - name: Download coverage artifacts
         uses: actions/download-artifact@v4
+        with:
+          path: coverage-artifacts
+      - name: Install lcov
+        run: sudo apt-get install -y lcov
       - name: Merge coverages
         run: |
-          sudo apt-get install -y "lcov"
-          LCOV_FILES="$(find . -name "lcov.info")"
-          CMD="$(which "lcov")"
-          for LCOV_FILE in ${LCOV_FILES}; do
-            CMD+=" -a ${LCOV_FILE}"
+          LCOV_FILES=$(find coverage-artifacts -name "lcov.info" -type f)
+          
+          if [ -z "$LCOV_FILES" ]; then
+            echo "No lcov.info files found in the artifacts!"
+            exit 1
+          fi
+          
+          CMD="lcov"
+          for LCOV_FILE in $LCOV_FILES; do
+            echo "Adding coverage file: $LCOV_FILE"
+            CMD+=" -a $LCOV_FILE"
           done
           CMD+=" -o lcov.info"
-          echo "Merging coverages: ${LCOV_FILES}"
-          ${CMD}
+          
+          echo "Executing: $CMD"
+          eval $CMD
+          
+          if [ ! -f "lcov.info" ]; then
+            echo "Failed to generate merged lcov.info file!"
+            exit 1
+          fi
+          
+          echo "Successfully merged $(echo $LCOV_FILES | wc -w) coverage files."
       - name: Report coverage
         uses: k1LoW/octocov-action@v1
         with: