From 701970db0be58b880e22cdafd0aada74cd9c47a4 Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 10:53:39 +0000 Subject: [PATCH 1/7] cr comments --- .github/workflows/python_unit_tests.yaml | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/.github/workflows/python_unit_tests.yaml b/.github/workflows/python_unit_tests.yaml index 88d45e8e..08537111 100644 --- a/.github/workflows/python_unit_tests.yaml +++ b/.github/workflows/python_unit_tests.yaml @@ -5,6 +5,9 @@ on: push: branches: [main, 'releases/**'] workflow_dispatch: + paths: + - '**.py' + - '.github/workflows/python_unit_tests.yaml' permissions: read-all @@ -12,7 +15,7 @@ jobs: python_unit_tests: name: python_unit_tests runs-on: ubuntu-24.04 - timeout-minutes: 10 + timeout-minutes: 5 steps: - uses: actions/checkout@v5 @@ -20,18 +23,18 @@ jobs: uses: actions/setup-python@v5 with: python-version: '3.14' + cache: 'pip' - name: Install dependencies run: | - # We only have 1 external dependency other than pytest for now, so + # We only have 1 external dependencies other than pytest for now, so # list them here # If this changes, we may want to switch to a dependencies file of # some format python -m pip install --upgrade pip - pip install pytest - pip install networkx - pip install PyYAML + pip install pytest networkx PyYAML - name: Test with pytest run: | + # Setup git for git_bdiff and get_git_sources testing git config --global user.name 'Testing' git config --global user.email 'Testing' pytest -vv From f6422b75e06f7b93c9ad37611a1d8056fa08adf2 Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 10:58:51 +0000 Subject: [PATCH 2/7] cr comments --- .github/workflows/python_unit_tests.yaml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/python_unit_tests.yaml b/.github/workflows/python_unit_tests.yaml index 08537111..cbf22c8e 100644 --- a/.github/workflows/python_unit_tests.yaml +++ b/.github/workflows/python_unit_tests.yaml @@ -2,12 +2,15 @@ name: Python Unit Tests on: pull_request: + paths: + - '**.py' + - '.github/workflows/python_unit_tests.yaml' push: + paths: + - '**.py' + - '.github/workflows/python_unit_tests.yaml' branches: [main, 'releases/**'] workflow_dispatch: - paths: - - '**.py' - - '.github/workflows/python_unit_tests.yaml' permissions: read-all From 49f0f908e28330d16145d6413f0c31eb91a252bd Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 11:06:00 +0000 Subject: [PATCH 3/7] remove cache --- .github/workflows/python_unit_tests.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/python_unit_tests.yaml b/.github/workflows/python_unit_tests.yaml index cbf22c8e..3faa022e 100644 --- a/.github/workflows/python_unit_tests.yaml +++ b/.github/workflows/python_unit_tests.yaml @@ -26,7 +26,6 @@ jobs: uses: actions/setup-python@v5 with: python-version: '3.14' - cache: 'pip' - name: Install dependencies run: | # We only have 1 external dependencies other than pytest for now, so From 359311f0d12b9fe939c552c185547dda2c1da1d3 Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 11:09:39 +0000 Subject: [PATCH 4/7] add requirements file --- .github/workflows/python_unit_tests.yaml | 9 +-------- requirements.txt | 2 ++ 2 files changed, 3 insertions(+), 8 deletions(-) create mode 100644 requirements.txt diff --git a/.github/workflows/python_unit_tests.yaml b/.github/workflows/python_unit_tests.yaml index 3faa022e..38e15d66 100644 --- a/.github/workflows/python_unit_tests.yaml +++ b/.github/workflows/python_unit_tests.yaml @@ -26,14 +26,7 @@ jobs: uses: actions/setup-python@v5 with: python-version: '3.14' - - name: Install dependencies - run: | - # We only have 1 external dependencies other than pytest for now, so - # list them here - # If this changes, we may want to switch to a dependencies file of - # some format - python -m pip install --upgrade pip - pip install pytest networkx PyYAML + cache: 'pip' - name: Test with pytest run: | # Setup git for git_bdiff and get_git_sources testing diff --git a/requirements.txt b/requirements.txt new file mode 100644 index 00000000..83b1a101 --- /dev/null +++ b/requirements.txt @@ -0,0 +1,2 @@ +PyYAML +networkx From 2fb6a999f1a9ac5476cd9470606b620cf0da7ce2 Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 11:12:06 +0000 Subject: [PATCH 5/7] add pytest --- requirements.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/requirements.txt b/requirements.txt index 83b1a101..262c811b 100644 --- a/requirements.txt +++ b/requirements.txt @@ -1,2 +1,3 @@ +pytest PyYAML networkx From 114aa1e8846960eb17689878b4ab4fa6a505933e Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 11:25:17 +0000 Subject: [PATCH 6/7] revert to no requrements --- .github/workflows/python_unit_tests.yaml | 8 ++++++++ requirements.txt | 3 --- 2 files changed, 8 insertions(+), 3 deletions(-) delete mode 100644 requirements.txt diff --git a/.github/workflows/python_unit_tests.yaml b/.github/workflows/python_unit_tests.yaml index 38e15d66..578ca314 100644 --- a/.github/workflows/python_unit_tests.yaml +++ b/.github/workflows/python_unit_tests.yaml @@ -27,6 +27,14 @@ jobs: with: python-version: '3.14' cache: 'pip' + - name: Install dependencies + run: | + # We only have 2 external dependencies other than pytest for now, so + # list them here + # If this changes, we may want to switch to a dependencies file of + # some format + python -m pip install --upgrade pip + pip install pytest networkx PyYAML - name: Test with pytest run: | # Setup git for git_bdiff and get_git_sources testing diff --git a/requirements.txt b/requirements.txt deleted file mode 100644 index 262c811b..00000000 --- a/requirements.txt +++ /dev/null @@ -1,3 +0,0 @@ -pytest -PyYAML -networkx From a890062062ce63c744ac1326958c3307cd973015 Mon Sep 17 00:00:00 2001 From: James Bruten <109733895+james-bruten-mo@users.noreply.github.com> Date: Tue, 17 Feb 2026 11:26:29 +0000 Subject: [PATCH 7/7] save file --- .github/workflows/python_unit_tests.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/python_unit_tests.yaml b/.github/workflows/python_unit_tests.yaml index 578ca314..e88d90ff 100644 --- a/.github/workflows/python_unit_tests.yaml +++ b/.github/workflows/python_unit_tests.yaml @@ -26,7 +26,6 @@ jobs: uses: actions/setup-python@v5 with: python-version: '3.14' - cache: 'pip' - name: Install dependencies run: | # We only have 2 external dependencies other than pytest for now, so