Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Travis improvements #607

Merged
merged 3 commits into from
Jan 31, 2018
Merged

Travis improvements #607

merged 3 commits into from
Jan 31, 2018

Conversation

chorrell
Copy link
Contributor

@chorrell chorrell commented Jan 7, 2018

This adds a bunch of minor improvements to the travis-ci config:

  • Install and use the latest version of docker (via the docker-ce package).
  • Explicitly set the distro to Trusty.
  • Set the language to node_js and with the latest LTS version installed. Prior to this the image would default to the ruby language which always seemed weird to me. As a bonus this means we don't need to install node for the doc check

SimenB

This comment was marked as off-topic.

@SimenB
Copy link
Member

SimenB commented Jan 8, 2018

Nice!

PeterDaveHello

This comment was marked as off-topic.

LaurentGoderre

This comment was marked as off-topic.

@chorrell
Copy link
Contributor Author

@nodejs/docker Is this good to merge now?

@SimenB
Copy link
Member

SimenB commented Jan 31, 2018

Yeah, go for it

@chorrell
Copy link
Contributor Author

Cool, thanks!

@chorrell chorrell merged commit 2553444 into nodejs:master Jan 31, 2018
@chorrell chorrell deleted the travis-improvements branch January 9, 2019 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants