Skip to content

Commit 4df9146

Browse files
authored
Merge pull request #1053 from OneSignal/fix/cd-ci_python2_issue
[CI/CD Fix] Builds missing python 2
2 parents de03bc5 + 216f842 commit 4df9146

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

.github/workflows/cd.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@ on:
55

66
jobs:
77
test_build_deploy:
8-
runs-on: ubuntu-latest
8+
runs-on: ubuntu-20.04
9+
container:
10+
image: python:2.7.18-buster
911

1012
strategy:
1113
matrix:
@@ -17,9 +19,8 @@ jobs:
1719
uses: actions/setup-node@v3
1820
with:
1921
node-version: ${{ matrix.node-version }}
20-
- uses: actions/setup-python@v4
21-
with:
22-
python-version: '2.7'
22+
- name: Install Yarn
23+
run: npm install -g yarn
2324
- name: "[Setup] Install dependencies"
2425
run: yarn
2526
- name: "[Test] Run all tests"

.github/workflows/ci.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@ on:
66

77
jobs:
88
test:
9-
runs-on: ubuntu-latest
9+
runs-on: ubuntu-20.04
10+
container:
11+
image: python:2.7.18-buster
1012

1113
strategy:
1214
matrix:
@@ -18,10 +20,8 @@ jobs:
1820
uses: actions/setup-node@v3
1921
with:
2022
node-version: ${{ matrix.node-version }}
21-
- name: Use Python 2.7
22-
uses: actions/setup-python@v4
23-
with:
24-
python-version: '2.7'
23+
- name: Install Yarn
24+
run: npm install -g yarn
2525
- name: "[Setup] Install dependencies"
2626
run: yarn
2727
- name: "[Test] Run linters"

0 commit comments

Comments
 (0)