From fbd4b6c3f6f7e8c8826f53c4cd8c97de190d53f0 Mon Sep 17 00:00:00 2001
From: Kieran Ryan <kierankilkenny@gmail.com>
Date: Tue, 31 Dec 2024 21:09:36 +0000
Subject: [PATCH] ci: drop redundant Python test workflow dependencies (#340)

- Drop pip, setuptools, twine, wheel installations before 'Run tests' step in test workflow
- Ignore *.egg-info/ build artefact in .gitignore
- Use long option names in workflows such as --requirement and --editable
---
 .github/workflows/release-pypi.yaml | 2 +-
 .github/workflows/test-python.yml   | 5 ++---
 python/.gitignore                   | 1 +
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/release-pypi.yaml b/.github/workflows/release-pypi.yaml
index 4d40ddf15..225b5a5da 100644
--- a/.github/workflows/release-pypi.yaml
+++ b/.github/workflows/release-pypi.yaml
@@ -35,4 +35,4 @@ jobs:
       - name: Publish package distributions to PyPI
         uses: pypa/gh-action-pypi-publish@release/v1
         with:
-          packages-dir: python/dist
\ No newline at end of file
+          packages-dir: python/dist
diff --git a/.github/workflows/test-python.yml b/.github/workflows/test-python.yml
index 3c6188df0..daeff7d41 100644
--- a/.github/workflows/test-python.yml
+++ b/.github/workflows/test-python.yml
@@ -50,9 +50,8 @@ jobs:
         run: python --version
       - name: Install Python package dependencies
         run: |
-          python -m pip install twine pip setuptools wheel
-          pip install -r requirements.txt
-          pip install -e .
+          pip install --requirement requirements.txt
+          pip install --editable .
       - name: Run tests
         run: pytest
       - name: Run Acceptance Tests
diff --git a/python/.gitignore b/python/.gitignore
index ad9c7960a..26c59035e 100644
--- a/python/.gitignore
+++ b/python/.gitignore
@@ -9,3 +9,4 @@ venv/
 .tox/
 .coverage
 .coverage.*
+*.egg-info/