Skip to content

Commit

Permalink
Test io.js on AppVeyor
Browse files Browse the repository at this point in the history
  • Loading branch information
@kkoopa
kkoopa committed Jan 22, 2015
1 parent e8eeb80 commit 5d161f7
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion appveyor.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,19 @@
# Test against these versions of Node.js.
environment:
matrix:
# node.js
- nodejs_version: "0.8"
- nodejs_version: "0.10"
- nodejs_version: "0.11"
# io.js
- nodejs_version: "1.0"

# Install scripts. (runs after repo cloning)
install:
# Get the latest stable version of Node 0.STABLE.latest
- npm install npm
- move node_modules npm
- ps: Update-NodeJsInstallation (Get-NodeJsLatestBuild $env:nodejs_version)
- ps: Install-Product node $env:nodejs_version

This comment has been minimized.

Sign in to view

Copy link
@mathiask88

mathiask88 Jan 22, 2015

Contributor

This will only install the pre-installed images, that are mostly behind the bleeding edge, so I think the removed line is preferable.

Also not using the built in NPM for io.js will result in build error because node-gyp bundled in NPM is not yet io.js compatible.
I recently updated the file on my project: https://github.com/mathiask88/node-snap7/blob/master/appveyor.yml

This comment has been minimized.

Sign in to view

Copy link
@kkoopa

kkoopa Jan 22, 2015

Author Collaborator

Ah, great. Thanks.

This comment has been minimized.

Sign in to view

Copy link
@mathiask88

mathiask88 Jan 22, 2015

Contributor

Updated a few seconds ago. There was a little error in the if conditions. I hope it will work now. But I think io.js has a buggy npm in version 1.0.3 that will throw an error under windows (nodejs/node#536). But this is resolved in the next release.
# Typical npm stuff.
- npm/.bin/npm install
- npm/.bin/npm run rebuild-tests
Expand Down

0 comments on commit 5d161f7

Please sign in to comment.