Run integration tests on PRs (#868)

* Run integration tests on PRs * Move integration tests to main workflow
pypa · Feb 5, 2022 · b9a355c · b9a355c
1 parent 37a3f87
commit b9a355c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 28 deletions.
diff --git a/.github/workflows/integration.yml b/.github/workflows/integration.yml
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -57,6 +57,21 @@ jobs:
           name: ${{ matrix.python-version }} - ${{ matrix.platform }}
           fail_ci_if_error: true
 
+  # Because the tests can be flaky, they shouldn't be required for merge, but
+  # it's still helpful to run them on PRs. See:
+  # https://github.com/pypa/twine/issues/684#issuecomment-703150619
+  integration:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-python@v2
+        with:
+          python-version: ${{ env.MIN_PYTHON_VERSION }}
+      - name: Install dependencies
+        run: python -m pip install tox
+      - name: Run tests
+        run: python -m tox -e integration
+
   docs:
     runs-on: ubuntu-latest
     steps:
@@ -73,6 +88,7 @@ jobs:
     needs:
       - lint
       - test
+      # Not requiring integration, because they can be flaky
       - docs
     if: github.event_name == 'push' && contains(github.ref, 'refs/tags/')
     runs-on: ubuntu-latest