Merge pull request #515 from lbluque/main

Upload coverage only on main
CederGroupHub · Feb 8, 2025 · 42e8d0e · 42e8d0e
2 parents 40dcfbc + 2bdf79e
commit 42e8d0e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -50,7 +50,7 @@ jobs:
           pytest --cov=smol --splits 8 --group ${{ matrix.split }} --durations-path tests/.test_durations tests
 
       - name: Upload coverage
-        if: ${{ matrix.config.python == '3.12' && github.event_name == 'push' }}
+        if: ${{ matrix.config.python == '3.12' && github.ref == 'refs/heads/main'}}
         uses: actions/upload-artifact@v4
         with:
           include-hidden-files: true
@@ -59,6 +59,7 @@ jobs:
           if-no-files-found: error
 
   coverage:
+    if: github.ref == 'refs/heads/main'
     runs-on: ubuntu-latest
     needs: test
     steps: