Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

Commit

Permalink
fix: drop node 10 support (#331)
Browse files Browse the repository at this point in the history
BREAKING CHANGE: this moves our `engines` support and testing matrix
forward to node12 LTS and above.
  • Loading branch information
wraithgar authored Oct 5, 2021
1 parent 00e2164 commit 852c6ca
Show file tree
Hide file tree
Showing 3 changed files with 290 additions and 119 deletions.
61 changes: 32 additions & 29 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,54 +1,57 @@
name: CI

on: [push, pull_request]
on:
pull_request:
push:
branches:
- main
- latest

jobs:
build:
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v2
with:
node-version: '16'
cache: npm
- run: npm i --prefer-online -g npm@latest
- run: npm ci
- run: npm run lint

test:
strategy:
fail-fast: false
matrix:
node-version: [10.1.x, 10.x, 12.0.x, 12.x, 14.0.x, 14.x, 15.x, 16.x]
node-version: [12.13.0, 12.x, 14.15.0, 14.x, 16.x]
platform:
- os: ubuntu-latest
shell: bash
- os: macos-latest
shell: bash
- os: windows-latest
shell: bash
- os: windows-latest
shell: cmd
- os: windows-latest
shell: powershell
fail-fast: false

runs-on: ${{ matrix.platform.os }}
defaults:
run:
shell: ${{ matrix.platform.shell }}

steps:
- name: Checkout Repository
uses: actions/[email protected]

- name: Use Nodejs ${{ matrix.node-version }}
uses: actions/setup-node@v1
- uses: actions/checkout@v2
- uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}

- name: use latest npm
run: npm i -g npm@latest

- name: Install dependencies
run: npm install

# Run for all environments
- name: Run Tap Tests (no pre/post scripts, no debug checks)
if: matrix.platform.os != 'ubuntu-latest' || matrix.node-version != '14.x'
run: npm run test-only -- -c -t0
cache: npm
- run: npm i --prefer-online -g npm@latest
- run: npm ci
# TODO -t0 will need to go into npm test when we standardize on template-oss
- run: npm test --ignore-scripts -t0
- run: npm ls -a
# TODO these will need to go into npm test when we standardize on template-oss
env:
ARBORIST_DEBUG: '0'
NODE_OPTIONS: --no-warnings

# Push coverage for specific environment
- name: Run Tap Tests (push coverage results, verify linting, debug checks)
if: matrix.platform.os == 'ubuntu-latest' && matrix.node-version == '14.x'
run: npm test -- -c -t0
env:
COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}
Loading

0 comments on commit 852c6ca

Please sign in to comment.