jonnyzzz / TeamCity.Node

Node.js, NPM and Phantom.JS runners for TeamCity
Apache License 2.0
308 stars 49 forks source link

Node.js NVM runner detect pre-configured NVM #24

Open jperry opened 10 years ago

jperry commented 10 years ago

If an agent already has nvm installed use that one instead or at least have the ability to state which one it should use.

DylanShea commented 10 years ago

Agreed. I updated by node plugin from 1.0.28 to 1.0.42 and was surprised by the requirement to add a "Node.js NVM Installer" runner step, for my working build configurations that had "Node.js NPM" build runners. I already had created my own method of having a contained NVM/Node.js solution, which I want to continue to use. See also Issue #27.

shedar commented 8 years ago

This feature will also allow to use windows agents with nvm. I guess it'll be easier then installing of windows version of nvm for windows agents. For now I use command line runner to switch node version, which is not a great sollution

jonnyzzz commented 8 years ago

@shedar would you like to provide pull request for this?

srizzling commented 7 years ago

Is this something that still be supported? Maybe by setting the NVM Home on the agent?