Enable cov vars only on the run step

Author: nicoddemus

.github/workflows/main.yml

@@ -16,11 +16,6 @@ on:
 
 jobs:
   build:
-    env:
-      _PYTEST_TOX_COVERAGE_RUN: "coverage run -m"
-      COVERAGE_PROCESS_START: ".coveragerc"
-      _PYTEST_TOX_EXTRA_DEP: "coverage-enable-subprocess"
-
     runs-on: ${{ matrix.os }}
 
     strategy:
@@ -127,16 +122,18 @@ jobs:
         python -m pip install --upgrade pip
         pip install tox coverage
 
-    - name: Test (no coverage)
+    - name: Test without coverage
       if: "matrix.skip_coverage"
-      env:
-        _PYTEST_TOX_COVERAGE_RUN: ""
       run: |
         echo "_PYTEST_TOX_COVERAGE_RUN = $_PYTEST_TOX_COVERAGE_RUN"
         tox -e ${{ matrix.tox_env }}
 
-    - name: Test
+    - name: Test with coverage
       if: "! matrix.skip_coverage"
+      env:
+        _PYTEST_TOX_COVERAGE_RUN: "coverage run -m"
+        COVERAGE_PROCESS_START: ".coveragerc"
+        _PYTEST_TOX_EXTRA_DEP: "coverage-enable-subprocess"
       run: |
         echo "_PYTEST_TOX_COVERAGE_RUN = $_PYTEST_TOX_COVERAGE_RUN"
         tox -e ${{ matrix.tox_env }}

tox.ini

@@ -26,7 +26,7 @@ passenv = USER USERNAME COVERAGE_* TRAVIS PYTEST_ADDOPTS TERM
 setenv =
     _PYTEST_TOX_DEFAULT_POSARGS={env:_PYTEST_TOX_POSARGS_LSOF:} {env:_PYTEST_TOX_POSARGS_XDIST:}
 
-    # Configuration to run with coverage similar to Travis/Appveyor, e.g.
+    # Configuration to run with coverage similar to CI, e.g.
     # "tox -e py37-coverage".
     coverage: _PYTEST_TOX_COVERAGE_RUN=coverage run -m
     coverage: _PYTEST_TOX_EXTRA_DEP=coverage-enable-subprocess