Skip to content

Commit

Permalink
chore: add github actions workflow config for windows (#3078)
Browse files Browse the repository at this point in the history
* chore: add github actions workflow config

* fix: use npm ci, npm run test

* chore: wee

* chore: this might be custom appveyor syntax

* chore: split funky commands into steps

* Try to fix line-break issue in Windows

* chore: skip test:leak on windows

* chore: remove appveyor.yml

* Align a little closer with Node example

See if this improves output

* Try to fix step description

* Remove obsolete workflow name

* Add workflow name again

* chore: tweak descriptions, rename workflow file, add node 6 comment
  • Loading branch information
shellscape committed Aug 26, 2019
1 parent b1747e4 commit cc5fd63
Show file tree
Hide file tree
Showing 4 changed files with 54 additions and 41 deletions.
32 changes: 32 additions & 0 deletions .github/workflows/node-windows.yml
@@ -0,0 +1,32 @@
name: Node

on: [push]

jobs:
build:

runs-on: windows-2019

strategy:
matrix:
node: [ '10', '6' ]

name: ${{ matrix.node }} (Windows)
steps:
- name: Configure git line-breaks
run: git config --global core.autocrlf false
- name: Checkout Commit
uses: actions/checkout@v1
- name: Setup Node
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node }}
- name: Check npm version
# This is only necessary until we dump Node 6
run: IF ${{ matrix.node }} LSS 8 npm i npm@5 -g
- name: Install dependencies
run: npm ci --ignore-scripts
- name: Run tests
run: npm test
env:
CI: true
32 changes: 0 additions & 32 deletions appveyor.yml

This file was deleted.

22 changes: 14 additions & 8 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 8 additions & 1 deletion package.json
Expand Up @@ -32,7 +32,13 @@
"test": "npm run test:all",
"test:all": "npm run test:only && npm run test:typescript && npm run test:leak",
"test:coverage": "nyc --reporter html mocha",
"test:leak": "npm i --silent --no-save weak@1 && node --expose-gc test/leak/index.js",
"test:leak": "cross-os test:leak:os",
"test:leak:os": {
"darwin": "npm run test:leak:nix",
"linux": "npm run test:leak:nix",
"win32": "echo 'Skipping test:leak on Windows'"
},
"test:leak:nix": "npm i --silent --no-save weak@1 && node --expose-gc test/leak/index.js",
"test:only": "mocha",
"test:quick": "mocha -b",
"test:typescript": "tsc --noEmit -p test/typescript && tsc --noEmit",
Expand Down Expand Up @@ -73,6 +79,7 @@
"codecov": "^3.5.0",
"console-group": "^0.3.3",
"core-js": "^3.2.1",
"cross-os": "^1.3.0",
"date-time": "^3.1.0",
"es5-shim": "^4.5.13",
"es6-shim": "^0.35.5",
Expand Down

0 comments on commit cc5fd63

Please sign in to comment.