Merge pull request #16 from clalarco/issue-15-skip-python-installation

Resolves #15 Allow to use an external python module

Author: marian-code

.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

.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}"
+      

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