Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(jenkins): run Jenkins builds on Node 4 (via nvm) #13568

Closed

Conversation

petebacondarwin
Copy link
Contributor

No description provided.

#!/bin/bash

# Install nvm for this shell
source ~/.nvm/nvm.sh
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does jenkins provide this (i.e. where does it come from)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is manually installed on the server.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we print the nvm/Node/npm versions used? Could be useful for debugging.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that it does do that automatically...

+ scripts/jenkins/set-node-version.sh
v4.2.3 is already installed.
Now using node v4.2.3 (npm v2.14.7)

source ~/.nvm/nvm.sh

# Use node.js at 4.2.x
nvm install 4.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is nvm install ... sufficient or do we also need nvm use ... ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm install is sufficient.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4 would be enough, the latest 4.x.y release is an LTS, it so happens it's 4.2.y in fact.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the node.js docs:

Minors: Minors are the addition and refinements of APIs or subsystems. They do not generally change APIs or introduce backwards compatible breaking changes, except where unavoidable. These are mostly completely additive releases.

I think it is safer to stick with a fixed minor release for automatic updates

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. It doesn't matter here anyway as there will be no Node.js 4.3.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@petebacondarwin petebacondarwin deleted the jenkins-hacks branch November 24, 2016 09:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants