Skip to content

Commit 6ac110d

Browse files
authored
Only upgrade GitHub Actions
@No9 Simplifying as you suggest.
1 parent f80bc8b commit 6ac110d

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

.github/workflows/push.yml

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,18 @@ jobs:
1616
fail-fast: false
1717
matrix:
1818
node:
19+
- version: 10.x
20+
- version: 12.x
1921
- version: 14.x
20-
- version: 16.x
21-
- version: 18.x
22+
- version: 15.x
2223
mirror: https://nodejs.org/download/nightly
23-
- version: 18.x
24+
- version: 15.x
2425
mirror: https://nodejs.org/download/v8-canary
2526
# os: [ubuntu-latest, macos-latest]
2627
# Temporarily disable MacOS until
2728
# https://github.com/nodejs/node/issues/32981 is fixed
2829
# TODO(mmarchini): test on 20.04 (need different lldb version)
29-
os: [ubuntu-20.04] # [ubuntu-18.04, ubuntu-20.04, ubuntu-22.04]
30+
os: [ubuntu-18.04, ubuntu-20.04]
3031
llvm: [ 8, 9 ]
3132
steps:
3233
- uses: actions/checkout@v3
@@ -47,19 +48,19 @@ jobs:
4748
npm install --llnode_build_addon=true --llnode_coverage=true
4849
- name: run tests
4950
run: TEST_LLDB_BINARY=`which lldb-${{ matrix.llvm }}` npm run nyc-test-all
50-
if: matrix.node.version != '18.x'
51+
if: matrix.node.version != '15.x'
5152
- name: run tests (nightly)
5253
run: TEST_LLDB_BINARY=`which lldb-${{ matrix.llvm }}` npm run nyc-test-all
53-
if: matrix.node.version == '18.x'
54+
if: matrix.node.version == '15.x'
5455
continue-on-error: true
5556
- name: prepare coverage
56-
if: startsWith(matrix.os, 'ubuntu-') && matrix.node.version != '18.x'
57+
if: startsWith(matrix.os, 'ubuntu-') && matrix.node.version != '15.x'
5758
run: |
5859
npm run coverage
5960
cat ./coverage-js.info > ./coverage.info
6061
cat ./coverage-cc.info >> ./coverage.info
6162
- name: Upload coverage report to Codecov
62-
uses: codecov/codecov-action@v1
63+
uses: codecov/codecov-action@v3
6364
with:
6465
file: ./coverage.info
6566
linter:
@@ -69,7 +70,7 @@ jobs:
6970
- name: Use Node.js LTS
7071
uses: actions/setup-node@v3
7172
with:
72-
node-version: 18.x
73+
node-version: 12.x
7374
- name: npm install, build, and test
7475
run: |
7576
sudo apt-get -qq update

0 commit comments

Comments
 (0)