Merge pull request #870 from stripe/anniel-coveralls-update

Move coveralls step to be conditionally executed under test
stripe · Aug 25, 2022 · fcf1832 · fcf1832
2 parents f5fb273 + 5cd4572
commit fcf1832
Showing 1 changed file with 5 additions and 14 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -111,20 +111,11 @@ jobs:
       - name: Test with pytest
         run: make test-gh-actions
 
-  coveralls:
-    name: Coveralls
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v2
-
-    - name: Start stripe-mock
-      run: docker run -d -p 12111-12112:12111-12112 stripe/stripe-mock && sleep 5
-
-    - name: Calculate and publish coverage
-      run: make coveralls
-      env:
-        python-version: "3.10"
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      - name: Calculate and publish coverage
+        run: make coveralls
+        if: matrix.python-version == '3.10'
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
   publish:
     name: Publish