Ensure reproducible development and test environment

treiher · Dec 17, 2023 · 92de867 · 92de867
1 parent cde4fc3
commit 92de867
Show file tree

Hide file tree

Showing 7 changed files with 1,786 additions and 167 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -3,11 +3,13 @@ name: tests
 on:
   push:
   schedule:
-    - cron: '0 2 * * *'
+    - cron: '0 2 * * 6'
 
 env:
   RUST_VERSION: "1.68.2"
   TRUNK_VERSION: "0.17.5"
+  POETRY_VERSION: "1.7.1"
+  POETRY_DYNAMIC_VERSIONING_VERSION: "1.2.0"
 
 jobs:
   check_frontend:
@@ -21,7 +23,7 @@ jobs:
           echo ~/.cargo/bin >> $GITHUB_PATH
       - name: Install dependencies
         run: |
-          pip install setuptools_scm
+          pipx install poetry==${{ env.POETRY_VERSION }} poetry-dynamic-versioning==${{ env.POETRY_VERSION }}
       - name: Check
         run: |
           make check_frontend
@@ -37,17 +39,14 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - name: Set up Python
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
-          cache: pip
-          cache-dependency-path: |
-            setup.cfg
+          cache: poetry
       - name: Install dependencies
         run: |
-          python -m pip install --upgrade pip
-          pip install --user -e .[devel]
-          echo "PYTHONPATH=$PWD" >> $GITHUB_ENV
+          pipx install poetry==${{ env.POETRY_VERSION }} poetry-dynamic-versioning==${{ env.POETRY_VERSION }}
+          poetry install
       - name: Check
         run: |
           make check_${{ matrix.target }}
@@ -60,15 +59,11 @@ jobs:
         python-version: ["3.9", "3.10", "3.11"]
     steps:
       - uses: actions/checkout@v3
-        with:
-          submodules: true
       - name: Set up Python
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
-          cache: pip
-          cache-dependency-path: |
-            setup.cfg
+          cache: poetry
       - name: Set up Rust
         run: |
           curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain ${{ env.RUST_VERSION }} --target wasm32-unknown-unknown --profile minimal
@@ -79,8 +74,7 @@ jobs:
       - name: Install dependencies
         run: |
           sudo apt install libarchive-tools
-          python -m pip install --upgrade pip
-          pip install build setuptools_scm
+          pipx install poetry==${{ env.POETRY_VERSION }} poetry-dynamic-versioning==${{ env.POETRY_VERSION }}
           cargo install --locked trunk@${{ env.TRUNK_VERSION }}
       - name: Test
         run: |
@@ -109,17 +103,14 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
-          cache: pip
-          cache-dependency-path: |
-            setup.cfg
+          cache: poetry
       - name: Install dependencies
         run: |
-          python -m pip install --upgrade pip
-          pip install --user -e .[devel]
-          echo "PYTHONPATH=$PWD" >> $GITHUB_ENV
+          pipx install poetry==${{ env.POETRY_VERSION }} poetry-dynamic-versioning==${{ env.POETRY_VERSION }}
+          poetry install
       - name: Test
         run: |
           make test_backend
@@ -132,12 +123,10 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - name: Set up Python ${{ env.python-version }}
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ env.python-version }}
-          cache: pip
-          cache-dependency-path: |
-            setup.cfg
+          cache: poetry
       - name: Set up Rust
         run: |
           curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain ${{ env.RUST_VERSION }} --target wasm32-unknown-unknown --profile minimal
@@ -148,11 +137,9 @@ jobs:
       - name: Install dependencies
         run: |
           sudo apt install libarchive-tools
-          python -m pip install --upgrade pip
-          pip install build setuptools_scm
-          pip install --user -e .[devel]
+          pipx install poetry==${{ env.POETRY_VERSION }} poetry-dynamic-versioning==${{ env.POETRY_VERSION }}
+          poetry install
           cargo install --locked trunk@${{ env.TRUNK_VERSION }}
-          echo "PYTHONPATH=$PWD" >> $GITHUB_ENV
       - name: Test
         run: |
           make test_e2e
diff --git a/Makefile b/Makefile
@@ -9,7 +9,7 @@ FRONTEND_FILES := index.css manifest.json service-worker.js valens-frontend.js v
 PACKAGE_FRONTEND_FILES := valens/frontend $(addprefix valens/frontend/,$(FRONTEND_FILES))
 BUILD_DIR := $(PWD)/build
 CONFIG_FILE := $(BUILD_DIR)/config.py
-VERSION ?= $(shell python3 -c "import setuptools_scm; print(setuptools_scm.get_version())")
+VERSION ?= $(shell poetry version -s)
 WHEEL ?= dist/valens-$(VERSION)-py3-none-any.whl
 
 export SQLALCHEMY_WARN_20=1
@@ -55,13 +55,13 @@ test_frontend:
 test_backend:
 	mkdir -p valens/frontend
 	touch $(addprefix valens/frontend/,$(FRONTEND_FILES))
-	python3 -m pytest -n$(shell nproc) -vv --cov=valens --cov-branch --cov-fail-under=100 --cov-report=term-missing:skip-covered tests/backend
+	poetry run pytest -n$(shell nproc) -vv --cov=valens --cov-branch --cov-fail-under=100 --cov-report=term-missing:skip-covered tests/backend
 
 test_installation: $(BUILD_DIR)/venv/bin/valens
 	$(BUILD_DIR)/venv/bin/valens --version
 
 test_e2e: $(BUILD_DIR)/venv/bin/valens
-	python3 -m pytest -n$(shell nproc) -vv --driver chrome --headless tests/e2e
+	poetry run pytest -n$(shell nproc) -vv --driver chrome --headless tests/e2e
 
 $(BUILD_DIR)/venv:
 	python3 -m venv $(BUILD_DIR)/venv
@@ -105,7 +105,7 @@ screenshots: $(PACKAGE_FRONTEND_FILES)
 dist: $(WHEEL)
 
 $(WHEEL): $(PACKAGE_FRONTEND_FILES)
-	python3 -m build
+	poetry build
 
 valens/frontend:
 	mkdir -p valens/frontend
@@ -126,7 +126,7 @@ run_frontend:
 	PATH=~/.cargo/bin:${PATH} trunk --config frontend/Trunk.toml serve --port 8000
 
 run_backend: $(CONFIG_FILE)
-	VALENS_CONFIG=$(CONFIG_FILE) flask --app valens --debug run -h 0.0.0.0
+	VALENS_CONFIG=$(CONFIG_FILE) poetry run flask --app valens --debug run -h 0.0.0.0
 
 $(CONFIG_FILE): $(BUILD_DIR)
 	valens config -d build

diff --git a/README.md b/README.md
@@ -107,6 +107,7 @@ http {
 The following software is required:
 
 - Python 3
+- [Poetry](https://python-poetry.org/) with [dynamic versioning plugin](https://github.com/mtkennerly/poetry-dynamic-versioning)
 - Rust toolchain
 - tmux (optional)
 
@@ -124,22 +125,10 @@ Install the Rust development tools.
 $ cargo install --locked trunk
 ```
 
-Create a Python virtual environment.
+Install the Python project and development tools.
 
 ```console
-$ python3 -m venv .venv
-```
-
-Activate the virtual environment.
-
-```console
-$ . .venv/bin/activate
-```
-
-Install the Python development tools and install the package in editable mode.
-
-```console
-$ pip install -e ".[devel]"
+$ poetry install
 ```
 
 ### Running development servers