From 70366d4f1081d2a030ca8eb52feff2976a2b6724 Mon Sep 17 00:00:00 2001
From: Fridolin Glatter <83776373+glatterf42@users.noreply.github.com>
Date: Wed, 6 Nov 2024 16:47:04 +0100
Subject: [PATCH] Upload coverage report from `main` directly (#125)

* Run whole test suite also on pushes to main
* Remove outdated build-docs workflow
---
 .github/workflows/build-docs.yaml | 65 -------------------------------
 .github/workflows/pytest.yaml     |  9 ++++-
 2 files changed, 7 insertions(+), 67 deletions(-)
 delete mode 100644 .github/workflows/build-docs.yaml

diff --git a/.github/workflows/build-docs.yaml b/.github/workflows/build-docs.yaml
deleted file mode 100644
index 57399cce..00000000
--- a/.github/workflows/build-docs.yaml
+++ /dev/null
@@ -1,65 +0,0 @@
-name: build the docs
-
-on:
-  push:
-    branches: ["main"]
-  pull_request:
-    branches: ["**"]
-
-jobs:
-  build-docs:
-    timeout-minutes: 5
-    runs-on: ubuntu-latest
-
-    steps:
-      #------------------------------------
-      #  check-out repo and set-up python
-      #------------------------------------
-      - name: Check out repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Set up python
-        id: setup-python
-        uses: actions/setup-python@v5
-        with:
-          python-version: "3.12"
-
-      #------------------------------
-      #  install & configure poetry
-      #------------------------------
-      - name: Install Poetry
-        uses: snok/install-poetry@v1
-        with:
-          virtualenvs-create: true
-          virtualenvs-in-project: true
-          installer-parallel: true
-
-      #----------------------------------
-      #  install dynamic version plugin
-      #----------------------------------
-      - name: Install poetry-dynamic-versioning
-        run: poetry self add "poetry-dynamic-versioning[plugin]"
-
-      #------------------------------------
-      #  load cached venv if cache exists
-      #------------------------------------
-      - name: Load cached venv
-        id: cached-poetry-dependencies
-        uses: actions/cache@v4
-        with:
-          path: .venv
-          key: venv-${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('**/poetry.lock') }}
-
-      #------------------------
-      #  install root project
-      #------------------------
-      - name: Install library
-        run: poetry install --no-interaction --with docs,dev,server
-
-      #------------------
-      #  build the docs
-      #------------------
-      - name: Build the docs
-        run: poetry run make --directory=doc html
diff --git a/.github/workflows/pytest.yaml b/.github/workflows/pytest.yaml
index 0d38dadf..cae82d03 100644
--- a/.github/workflows/pytest.yaml
+++ b/.github/workflows/pytest.yaml
@@ -1,6 +1,12 @@
 name: test
 
-on: pull_request
+# TODO workaround for Codecov/squash and merge, a bit wasteful
+# Revise e.g. via https://github.com/codecov/feedback/issues/369
+on: 
+  pull_request:
+  push:
+    branches:
+      - 'main'
 
 jobs:
   test:
@@ -130,7 +136,6 @@ jobs:
         run: |
           source .venv/bin/activate
           pytest --backend ${{ matrix.backend }} --postgres-dsn "postgresql://postgres:postgres@localhost:5432/test" --cov-report xml:.coverage.xml --cov=ixmp4 -rsxX --benchmark-skip --color=yes
-
       #------------------------------
       #  Upload coverage to codecov
       #------------------------------