Skip to content

Commit

Permalink
Run Python auto package installation CI job for Python 2 on Windows 2019
Browse files Browse the repository at this point in the history
Python 2 isn't supported on Windows Server 2022, which is now
`windows-latest`.
  • Loading branch information
henrymercer committed Feb 8, 2022
1 parent b8047e5 commit 2ebc770
Showing 1 changed file with 20 additions and 14 deletions.
34 changes: 20 additions & 14 deletions .github/workflows/python-deps.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Test Python Package Installation on Linux and Mac
name: Test Python Package Installation

on:
push:
Expand All @@ -12,11 +12,11 @@ jobs:
test-setup-python-scripts:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
python_deps_type: [pipenv, poetry, requirements, setup_py]
python_version: [2, 3]
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
python_deps_type: [pipenv, poetry, requirements, setup_py]
python_version: [2, 3]

env:
PYTHON_DEPS_TYPE: ${{ matrix.python_deps_type }}
Expand Down Expand Up @@ -64,9 +64,9 @@ jobs:
test-setup-python-scripts-non-standard-location:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]

steps:
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
Expand Down Expand Up @@ -108,12 +108,18 @@ jobs:
test -z $LGTM_INDEX_IMPORT_PATH
test-setup-python-scripts-windows:
runs-on: windows-latest
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
python_deps_type: [pipenv, poetry, requirements, setup_py]
python_version: [2, 3]
fail-fast: false
matrix:
python_deps_type: [pipenv, poetry, requirements, setup_py]
python_version: [2, 3]
include:
- python_version: 3
os: windows-latest
# Python 2 isn't supported on the `windows-latest` image, so use `windows-2019`.
- python_version: 2
os: windows-2019

env:
PYTHON_DEPS_TYPE: ${{ matrix.python_deps_type }}
Expand Down

0 comments on commit 2ebc770

Please sign in to comment.