Skip to content

Commit a33d16c

Browse files
hashtagchrisjoshmgross
authored andcommitted
Use separate steps for install, build and test (actions#192)
* Use separate steps for install, build and test Having separate steps makes it easier to determine what failed. Switching from `npm ci` to `npm install` because the Windows image uses node v8.10.0 and npm v5.6.0. `ci` was introduced in npm v5.7.0. See actions/runner-images#88 * Remove `name`
1 parent 2ce3a0e commit a33d16c

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

ci/node.js.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,8 @@ jobs:
1717
uses: actions/setup-node@v1
1818
with:
1919
node-version: ${{ matrix.node-version }}
20-
- name: npm install, build, and test
21-
run: |
22-
npm ci
23-
npm run build --if-present
24-
npm test
20+
- run: npm install
21+
- run: npm run build --if-present
22+
- run: npm test
2523
env:
2624
CI: true

0 commit comments

Comments
 (0)