FYI: Node.js 5.0.0 is out

m.goleb's Avatar

m.goleb

29 Oct, 2015 10:21 PM

Node.js 5.0.0 is out: https://nodejs.org/en/blog/release/v5.0.0/

Please add it to CI. :)

  1. 1 Posted by xzyfer on 03 Nov, 2015 11:42 PM

    xzyfer's Avatar

    +1 we depend on this for release native node binaries which currently cannot support Node 5 on Windows.

  2. Support Staff 2 Posted by Feodor Fitsner on 04 Nov, 2015 12:25 AM

    Feodor Fitsner's Avatar

    But it's already available on all build workers. Use this cmdlet to switch:

    Install-Product node ''
    
  3. 3 Posted by xzyfer on 04 Nov, 2015 01:12 AM

    xzyfer's Avatar

    I don't believe this is applicable because we test under multiple versions of Node.
    We're using the documented method for doing so - http://www.appveyor.com/docs/lang/nodejs-iojs#testing-under-multiple-versions-of-node-js-or-io-js

    However `nodejs_version: "5"` does not use Node 5.

    https://github.com/sass/node-sass/blob/master/appveyor.yml#L22-L30

  4. Support Staff 4 Posted by Feodor Fitsner on 04 Nov, 2015 01:14 AM

    Feodor Fitsner's Avatar

    Can you drop a link to a build please?

  5. 5 Posted by xzyfer on 04 Nov, 2015 01:20 AM

    xzyfer's Avatar

    This is the latest build - https://ci.appveyor.com/project/sass/node-sass/build/1410

    (ignore the failure)

  6. Support Staff 6 Posted by Feodor Fitsner on 04 Nov, 2015 01:21 AM

    Feodor Fitsner's Avatar
  7. 7 Posted by Michael Mifsud on 04 Nov, 2015 01:44 AM

    Michael Mifsud's Avatar

    Interesting. How is that possible given that commit is the head of master and the appveyor config change happened days beforehand?

  8. 8 Posted by m.goleb on 04 Nov, 2015 01:50 AM

    m.goleb's Avatar

    I think he says the second part of the config doesn't contain Node 5 and
    that this part was used, not the "release" part above. The commit is not
    relevant to this remark.

  9. 9 Posted by xzyfer on 04 Nov, 2015 01:56 AM

    xzyfer's Avatar

    I see. I misread the PR and assumed we were using Node 5 for both. This should be patched on our end. I can confirm release does build Node 5. Thank you both

    https://ci.appveyor.com/project/sass/node-sass/build/1411

  10. m.goleb closed this discussion on 29 Dec, 2015 01:27 PM.

Comments are currently closed for this discussion. You can start a new one.

Keyboard shortcuts

Generic

? Show this help
ESC Blurs the current field

Comment Form

r Focus the comment reply box
^ + ↩ Submit the comment

You can use Command ⌘ instead of Control ^ on Mac