simplifying CI

Author: gilesknap

.github/workflows/code.yml

@@ -110,8 +110,6 @@ jobs:
         run: |
           docker run --name test build bash /project/.github/workflows/container_tests.sh
           docker cp test:/project/dist .
-          docker cp test:/project/requirements.txt .
-          docker cp test:/project/requirements_dev.txt .
           docker cp test:/project/cov.xml .
 
       - name: Upload coverage to Codecov
@@ -134,11 +132,8 @@ jobs:
       - name: Upload build files
         uses: actions/upload-artifact@v3
         with:
-          name: build_files
-          path: |
-            dist
-            requirements.txt
-            requirements_dev.txt
+          name: dist
+          path: dist/*
 
   sdist:
     needs: container
@@ -151,7 +146,7 @@ jobs:
         # ${GITHUB_REPOSITORY##*/} is the repo name without org
         # Replace this with the cli command if different to the repo name
         run: |
-          pip install build_files/dist/*.gz
+          pip install dist/*.gz
           ${GITHUB_REPOSITORY##*/} --version
 
   release:
@@ -169,14 +164,13 @@ jobs:
         uses: softprops/action-gh-release@1e07f4398721186383de40550babbdf2b84acfc5 # v0.1.14
         with:
           files: |
-            build_files/*
-            build_files/dist/*
+            dist/*
           generate_release_notes: true
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Publish to PyPI
         env:
           TWINE_USERNAME: __token__
-          TWINE_PASSWORD: ${{ secrets.pypi_token }}
-        run: pipx run twine upload build_files/dist/*
+          TWINE_PASSWORD: ${{ secrets.PYPI_TOKEN }}
+        run: pipx run twine upload dist/*/whl dist/*.tar.gz

.github/workflows/container_tests.sh

@@ -6,7 +6,7 @@ source /venv/bin/activate
 
 touch requirements_dev.txt
 pip install -r requirements_dev.txt -e .[dev]
-pip freeze --exclude-editable > requirements_dev.txt
+pip freeze --exclude-editable > dist/requirements_dev.txt
 
 pipdeptree
 

.github/workflows/docs.yml

@@ -9,7 +9,6 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: ["ubuntu-latest"]
         python: ["3.10"]
 
     runs-on: ubuntu-latest

.github/workflows/linkcheck.yml

@@ -10,7 +10,6 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: ["ubuntu-latest"]
         python: ["3.10"]
 
     runs-on: ubuntu-latest

Dockerfile

@@ -30,10 +30,10 @@ ENV PATH=/venv/bin:$PATH
 RUN cd /project && \
     pip install --upgrade pip && \
     pip install -r requirements.txt dist/*.whl && \
-    pip freeze  > requirements.txt && \
+    pip freeze  > dist/requirements.txt && \
     # we don't want to include our own wheel in requirements - remove with sed
     # and replace with a comment to avoid a zero length asset upload later
-    sed -i '/file:/s/^/# Requirements for /' requirements.txt
+    sed -i '/file:/s/^/# Requirements for /' dist/requirements.txt
 
 FROM python:3.10-slim as runtime
 

setup.cfg

@@ -33,18 +33,16 @@ setup_requires =
 # For development tests/docs
 dev =
     black==22.6.0
-    pipdeptree
     flake8-isort
     isort>5.0
     mypy
+    pipdeptree
     pre-commit
     pytest-cov
+    setuptools_scm[toml]>=6.2
     sphinx-rtd-theme-github-versions
     tox
-    setuptools_scm[toml]>=6.2
     types-mock
-    sphinx-rtd-theme-github-versions
-    sphinx-apischema
 
 [options.packages.find]
 where = src