From 99fbd8921018fc177b5ac1d8db461f48af5d1544 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Claudio=20Alarc=C3=B3n=20Reyes?= <clalarco@gmail.com>
Date: Sat, 16 Nov 2024 07:11:14 -0300
Subject: [PATCH 1/2] Using different macos versions for python older and newer
 than 3.11

---
 .github/workflows/all-lints.yml | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/all-lints.yml b/.github/workflows/all-lints.yml
index ceea1c6..72071e5 100644
--- a/.github/workflows/all-lints.yml
+++ b/.github/workflows/all-lints.yml
@@ -8,8 +8,20 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-latest, macos-latest, windows-latest]
+        os: [ubuntu-latest, macos-13, macos-latest, windows-latest]
         python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
+        exclude:
+          - os: macos-latest
+            python-version: 3.8
+          - os: macos-latest
+            python-version: 3.9
+          - os: macos-latest
+            python-version: 3.10
+          - os: macos-13
+            python-version: 3.11
+          - os: macos-13
+            python-version: 3.12
+
     steps:
     - uses: actions/checkout@v4
     - uses: marian-code/python-lint-annotate@master

From 93ec7a85c2919b649558edfff68099475d940c36 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Claudio=20Alarc=C3=B3n=20Reyes?= <clalarco@gmail.com>
Date: Wed, 13 Nov 2024 10:16:03 -0300
Subject: [PATCH 2/2] ISSUE-15 New parameter to use an existing python
 installation.

- New parameter use-external-python to use an already installed python
- Installation of requirements.txt uses --ignore-installed to avoid overriding versions defined in an external python
- Adding a test workflow to check use-external-python
---
 .github/workflows/test-python-install.yml | 55 +++++++++++++++++++++++
 action.yml                                | 10 ++++-
 2 files changed, 63 insertions(+), 2 deletions(-)
 create mode 100644 .github/workflows/test-python-install.yml

diff --git a/.github/workflows/test-python-install.yml b/.github/workflows/test-python-install.yml
new file mode 100644
index 0000000..038dcd5
--- /dev/null
+++ b/.github/workflows/test-python-install.yml
@@ -0,0 +1,55 @@
+on:
+  push:
+  pull_request:
+name: Check python installation
+jobs:
+  pythoninstall:
+    name: Test Python install
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest]
+        python-version: ['3.11']
+        use-external-python: [true, false]
+    env:
+      python-test-package: python-dummy
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Install Python if required
+      if: ${{ matrix.use-external-python }}
+      uses: actions/setup-python@v5
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - name: Install a test dependency if external Python is used
+      if: ${{ matrix.use-external-python }}
+      run:
+        pip install ${{ env.python-test-package }}
+
+    - name: Test Action usage
+      uses: ./
+      with:
+        python-root-list: "./tests/*.py ./tests/subtest/*.py"
+        use-black: true
+        use-isort: true
+        use-mypy: true
+        use-pycodestyle: true
+        use-pydocstyle: true
+        extra-pycodestyle-options: "--max-line-length=88"
+        use-pylint: false
+        use-flake8: false
+        use-vulture: true
+        python-version: ${{ matrix.python-version }}
+        use-external-python: ${{ matrix.use-external-python }}
+
+    - name: Check if test dependency exists after execution
+      run: |
+        pip freeze > all-deps.txt
+        should_appear=$( if [[ "${{ matrix.use-external-python }}" == "true" ]]; then echo 0; else echo 1; fi )
+        line_exists=$( grep -qF "${{ env.python-test-package }}" "all-deps.txt"; echo $? )
+        echo "test package should be installed: ${{ matrix.use-external-python }}"
+        echo "test package is present (0 = present): ${line_exists}"
+        cat all-deps.txt
+        test "${should_appear}" == "${line_exists}"
+      
diff --git a/action.yml b/action.yml
index 74232d0..746853d 100644
--- a/action.yml
+++ b/action.yml
@@ -76,11 +76,17 @@ inputs:
     description: "Set desired python version with this keyword"
     required: false
     default: "3.8"
+  use-external-python:
+    description: "false (default): Install a new Python for this action; true: use the python installation in the previous steps"
+    type: boolean
+    required: false
+    default: false
 
 runs:
   using: "composite"
   steps:
     - name: Setup python
+      if: ${{ ! inputs.use-external-python }}
       uses: actions/setup-python@v5
       with:
         python-version: ${{ inputs.python-version }}
@@ -93,13 +99,13 @@ runs:
     - name: Windows install dependencies
       if: ${{ runner.os == 'Windows' }}
       run:  |
-        pip install -r ${{ github.action_path }}\requirements.txt
+        pip install -r ${{ github.action_path }}\requirements.txt --ignore-installed
         echo "path_sep=" >> $GITHUB_ENV
       shell: pwsh
 
     - name: Posix install dependencies
       if: ${{ runner.os != 'Windows' }}
-      run:  pip install -r ${{ github.action_path }}/requirements.txt
+      run:  pip install -r ${{ github.action_path }}/requirements.txt --ignore-installed
       shell: bash
 
     - name: Lint on Windows