From 0b4d5ce5e34ab46b5b55976bfdd0d1d0b105cf13 Mon Sep 17 00:00:00 2001 From: Erik Johnston Date: Fri, 8 Oct 2021 10:05:48 +0100 Subject: [PATCH] Fix CI to run the unit tests without optional deps (#11017) This also turns off calculating code coverage, as we didn't use it and it was a lot of noise --- .github/workflows/tests.yml | 9 ++++++--- changelog.d/11017.misc | 1 + 2 files changed, 7 insertions(+), 3 deletions(-) create mode 100644 changelog.d/11017.misc diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 96c39dd9a4..30a911fdbd 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -78,20 +78,23 @@ jobs: matrix: python-version: ["3.6", "3.7", "3.8", "3.9", "3.10"] database: ["sqlite"] + toxenv: ["py"] include: # Newest Python without optional deps - python-version: "3.10" - toxenv: "py-noextras,combine" + toxenv: "py-noextras" # Oldest Python with PostgreSQL - python-version: "3.6" database: "postgres" postgres-version: "9.6" + toxenv: "py" # Newest Python with newest PostgreSQL - python-version: "3.10" database: "postgres" postgres-version: "14" + toxenv: "py" steps: - uses: actions/checkout@v2 @@ -111,7 +114,7 @@ jobs: if: ${{ matrix.postgres-version }} timeout-minutes: 2 run: until pg_isready -h localhost; do sleep 1; done - - run: tox -e py,combine + - run: tox -e ${{ matrix.toxenv }} env: TRIAL_FLAGS: "--jobs=2" SYNAPSE_POSTGRES: ${{ matrix.database == 'postgres' || '' }} @@ -169,7 +172,7 @@ jobs: with: python-version: ${{ matrix.python-version }} - run: pip install tox - - run: tox -e py,combine + - run: tox -e py env: TRIAL_FLAGS: "--jobs=2" - name: Dump logs diff --git a/changelog.d/11017.misc b/changelog.d/11017.misc new file mode 100644 index 0000000000..f05530ac94 --- /dev/null +++ b/changelog.d/11017.misc @@ -0,0 +1 @@ +Fix CI to run the unit tests without optional deps.