Skip to content

Commit

Permalink
ci: update environment
Browse files Browse the repository at this point in the history
- Enable Node.js v17 testing
- Upgrade baseline Node.js to v16
- Re-enable Node.js v8.11.1 testing
- Remove unecessary configuration

see: loopbackio/cicd#2
see: loopbackio/cicd#4

Signed-off-by: Rifa Achrinza <[email protected]>
  • Loading branch information
achrinza committed Dec 11, 2021
1 parent c3a9ebd commit b6724c9
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 13 deletions.
21 changes: 9 additions & 12 deletions .github/workflows/continuous-integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,27 +19,28 @@ jobs:
strategy:
matrix:
os: [ubuntu-latest]
node-version: [10, 12, 14, 16]
node-version: [8.11.1, 10, 12, 14, 16, 17]
include:
- os: macos-latest
node-version: 14 # LTS
node-version: 16 # LTS
- os: windows-latest
node-version: 14 # LTS
node-version: 16 # LTS
fail-fast: false
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
- name: Update NPM (Node.js v8.11.1)
if: matrix.node-version != '8.11.1'
run: npm install --global npm@6
- name: Update NPM (Node.js v10)
if: matrix.node-version == 10
run: npm install --global npm@7
- name: Update NPM
if: matrix.node-version != 10
if: contains(fromJSON('[10, "8.11.1"]'), matrix.node-version )
run: npm install --global npm@8
- name: Bootstrap project
if: ${{ matrix.node-version != '8.11.1' }}
Expand All @@ -51,10 +52,6 @@ jobs:
run: npm run --ignore-scripts build
- name: Run tests
run: npm test --ignore-scripts
- name: troubleshooting
run: |
ls
ls ./coverage
- name: Coveralls Parallel
uses: coverallsapp/github-action@master
with:
Expand Down Expand Up @@ -82,10 +79,10 @@ jobs:
- uses: actions/checkout@v2
with:
fetch-depth: 0
- name: Use Node.js 14
- name: Use Node.js 16
uses: actions/setup-node@v2
with:
node-version: 14
node-version: 16
- name: Bootstrap project
run: npm ci --ignore-scripts
- name: Verify commit linting
Expand Down
7 changes: 6 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,18 @@ branches:
- master
language: node_js
node_js:
- 8.11.1
- 10
- 12
- 14
- 16
- 17
before_install: |
NODEJS_VERSION=$(node --version)
if [ 'v10' == ${NODEJS_VERSION%%.*} ]
if [ 'v8' == ${NODEJS_VERSION%%.*} ]
then
npm install --global npm@6
elif [ 'v10' == ${NODEJS_VERSION%%.*} ]
then
npm install --global npm@7
else
Expand Down

0 comments on commit b6724c9

Please sign in to comment.