-
-
Notifications
You must be signed in to change notification settings - Fork 45
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
build: update Node.js versions and pin dependencies (#48)
* PR: #48 * use nyc instead of istanbul and fix npm config for newer node versions also removed node 0.6 from ci action becuase it can not be installed on ubuntu 20 * add missing node versions to ci action
- Loading branch information
Showing
2 changed files
with
49 additions
and
31 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,11 +6,10 @@ on: | |
|
||
jobs: | ||
test: | ||
runs-on: ubuntu-18.04 | ||
runs-on: ubuntu-20.04 | ||
strategy: | ||
matrix: | ||
name: | ||
- Node.js 0.6 | ||
- Node.js 0.8 | ||
- Node.js 0.10 | ||
- Node.js 0.12 | ||
|
@@ -31,61 +30,60 @@ jobs: | |
- Node.js 15.x | ||
- Node.js 16.x | ||
- Node.js 17.x | ||
- Node.js 18.x | ||
- Node.js 19.x | ||
- Node.js 20.x | ||
- Node.js 21.x | ||
|
||
include: | ||
- name: Node.js 0.6 | ||
node-version: "0.6" | ||
npm-i: [email protected] | ||
npm-rm: istanbul | ||
|
||
- name: Node.js 0.8 | ||
node-version: "0.8" | ||
npm-i: [email protected] | ||
npm-rm: istanbul | ||
npm-rm: nyc | ||
|
||
- name: Node.js 0.10 | ||
node-version: "0.10" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 0.12 | ||
node-version: "0.12" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: io.js 1.x | ||
node-version: "1.8" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: io.js 2.x | ||
node-version: "2.5" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: io.js 3.x | ||
node-version: "3.3" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 4.x | ||
node-version: "4.9" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 5.x | ||
node-version: "5.12" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 6.x | ||
node-version: "6.15" | ||
npm-i: [email protected] | ||
node-version: "6.17" | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 7.x | ||
node-version: "7.10" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 8.x | ||
node-version: "8.14" | ||
npm-i: [email protected] | ||
node-version: "8.17" | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 9.x | ||
node-version: "9.11" | ||
npm-i: [email protected] | ||
npm-i: [email protected] [email protected] | ||
|
||
- name: Node.js 10.x | ||
node-version: "10.24" | ||
|
@@ -102,7 +100,7 @@ jobs: | |
node-version: "13.14" | ||
|
||
- name: Node.js 14.x | ||
node-version: "14.18" | ||
node-version: "14.19" | ||
|
||
- name: Node.js 15.x | ||
node-version: "15.14" | ||
|
@@ -111,7 +109,19 @@ jobs: | |
node-version: "16.13" | ||
|
||
- name: Node.js 17.x | ||
node-version: "17.2" | ||
node-version: "17.4" | ||
|
||
- name: Node.js 18.x | ||
node-version: "18.20" | ||
|
||
- name: Node.js 19.x | ||
node-version: "19.9" | ||
|
||
- name: Node.js 20.x | ||
node-version: "20.12" | ||
|
||
- name: Node.js 21.x | ||
node-version: "21.7" | ||
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
|
@@ -139,7 +149,15 @@ jobs: | |
dirname "$(nvm which ${{ matrix.node-version }})" >> "$GITHUB_PATH" | ||
- name: Configure npm | ||
run: npm config set shrinkwrap false | ||
run: | | ||
if [[ "$(npm config get package-lock)" == "true" ]]; then | ||
npm config set package-lock false | ||
else | ||
npm config set shrinkwrap false | ||
fi | ||
- name: Remove non-test npm modules | ||
run: npm rm --silent --save-dev csv-parse raw-body stream-to-array | ||
|
||
- name: Remove npm module(s) ${{ matrix.npm-rm }} | ||
run: npm rm --silent --save-dev ${{ matrix.npm-rm }} | ||
|
@@ -176,7 +194,7 @@ jobs: | |
- name: Run tests | ||
shell: bash | ||
run: | | ||
if npm -ps ls istanbul | grep -q istanbul; then | ||
if npm -ps ls nyc | grep -q nyc; then | ||
npm run test-ci | ||
else | ||
npm test | ||
|
@@ -188,7 +206,7 @@ jobs: | |
|
||
- name: Collect code coverage | ||
uses: coverallsapp/github-action@master | ||
if: steps.list_env.outputs.istanbul != '' | ||
if: steps.list_env.outputs.nyc != '' | ||
with: | ||
github-token: ${{ secrets.GITHUB_TOKEN }} | ||
flag-name: run-${{ matrix.test_number }} | ||
|
@@ -201,5 +219,5 @@ jobs: | |
- name: Upload code coverage | ||
uses: coverallsapp/github-action@master | ||
with: | ||
github-token: ${{ secrets.github_token }} | ||
github-token: ${{ secrets.GITHUB_TOKEN }} | ||
parallel-finished: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters