Merge pull request #1890 from minrk/coverage-bump

bump coverage in test env
zeromq · Jul 27, 2023 · 588627e · 588627e
2 parents 8e90224 + 3bec21d
commit 588627e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -105,10 +105,11 @@ jobs:
           grep -v plugins .coveragerc > .coveragerc-save
           mv .coveragerc-save .coveragerc
 
-      - name: enable Cython coverage
-        if: "! (startsWith(matrix.python, 'pypy') || startsWith(matrix.python, '3.12'))"
-        run: |
-          echo "PYZMQ_CYTHON_COVERAGE=1" >> "$GITHUB_ENV"
+      # - name: enable Cython coverage
+      #   if: "! (startsWith(matrix.python, 'pypy') || startsWith(matrix.python, '3.12'))"
+      #   run: |
+      #     echo "PYZMQ_CYTHON_COVERAGE=1" >> "$GITHUB_ENV"
+
       # preserve pip cache to speed up installation
       - name: Cache pip
         uses: actions/cache@v3

diff --git a/test-requirements.txt b/test-requirements.txt
@@ -2,7 +2,8 @@ black; platform_python_implementation != "PyPy"
 # gevent wheels on mac cause Illegal Instruction
 codecov
 # coverage 5 has issues with Cython: https://github.com/cython/cython/issues/3515
-coverage<5
+coverage<5; python_version < "3.7"
+coverage>=7.1; python_version >= "3.7"
 cython; platform_python_implementation != "PyPy" # required for Cython tests
 cython>=3.0.0b3; platform_python_implementation != "PyPy" and python_version >= "3.12" # required for Cython tests
 flake8