improve container_tests

Author: gilesknap

.github/workflows/container_tests.sh

@@ -2,17 +2,12 @@
 set -x
 
 cd /project
-
-python -m venv /tmp/venv
-source /tmp/venv/bin/activate
+source /venv/bin/activate
 
 touch requirements_dev.txt
 pip install -r requirements_dev.txt -e .[dev]
 pip freeze --exclude-editable > requirements_dev.txt
 
 pipdeptree
 
-# ensure non-zero length requirements.txt
-echo "# runtime dependencies" >> requirements.txt
-
 pytest tests

Dockerfile

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