Merge pull request childmindresearch#77 from childmindresearch/maint/ci-coverage

Fix coverage commenting from pull requests

Author: kaitj

.github/workflows/ci.yaml

@@ -2,13 +2,12 @@ name: CI
 
 permissions:
   contents: read
-  pull-requests: write
 
 on:
   push:
-    branches: [ "main" ]
+    branches: ["main"]
   pull_request:
-    branches: [ "main" ]
+    branches: ["main"]
 
 env:
   UV_FROZEN: true
@@ -33,7 +32,7 @@ jobs:
     needs: format
     strategy:
       matrix:
-        python-version: [ "3.11", "3.12", "3.13", "3.14" ]
+        python-version: ["3.11", "3.12", "3.13", "3.14"]
     steps:
       - uses: actions/checkout@v6
         with:
@@ -68,10 +67,16 @@ jobs:
             pytest-cloud.xml \
             pytest.xml
 
-      - name: Pytest coverage comment
-        if: github.event_name == 'pull_request' && matrix.python-version ==
-          env.COVERAGE_PYTHON
-        uses: MishaKav/pytest-coverage-comment@v1
+      - name: Record PR number
+        if: github.event_name == 'pull_request' && matrix.python-version == env.COVERAGE_PYTHON
+        run: echo "${{ github.event.number }}" > pr-number.txt
+
+      - uses: actions/upload-artifact@v7
+        if: github.event_name == 'pull_request' && matrix.python-version == env.COVERAGE_PYTHON
         with:
-          pytest-xml-coverage-path: ./coverage.xml
-          junitxml-path: ./pytest.xml
+          name: coverage
+          retention-days: 1
+          path: |
+            coverage.xml
+            pytest.xml
+            pr-number.txt

.github/workflows/coverage.yaml

@@ -0,0 +1,33 @@
+name: Coverage report
+
+on:
+  workflow_run:
+    workflows: ["CI"]
+    types: [completed]
+
+permissions:
+  pull-requests: write
+  actions: read
+
+jobs:
+  coverage:
+    if: >-
+      github.event.workflow_run.event == 'pull_request' &&
+      github.event.workflow_run.conclusion == 'success'
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/download-artifact@v8
+        with:
+          name: coverage
+          run-id: ${{ github.event.workflow_run.id }}
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+
+      - id: pr
+        run: echo "number=$(cat pr-number.txt)" >> "$GITHUB_OUTPUT"
+
+      - name: Pytest coverage comment
+        uses: MishaKav/pytest-coverage-comment@v1
+        with:
+          issue-number: ${{ steps.pr.outputs.number }}
+          pytest-xml-coverage-path: ./coverage.xml
+          junitxml-path: ./pytest.xml