Skip to content

Update Travis config#29

Merged
lttb merged 1 commit intomasterfrom
feature/travis-ci
May 31, 2017
Merged

Update Travis config#29
lttb merged 1 commit intomasterfrom
feature/travis-ci

Conversation

@lttb
Copy link
Copy Markdown
Member

@lttb lttb commented May 31, 2017

No description provided.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 978ee73 on feature/travis-ci into d3a578a on master.

@lttb lttb merged commit d227e5a into master May 31, 2017
@lttb lttb deleted the feature/travis-ci branch May 31, 2017 13:41
Comment thread .travis.yml
@@ -1,6 +1,12 @@
dist: trusty
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

whats that?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Travis uses Ubuntu Precise as a default image, but with this option Ubuntu Trusty will be used. So official support of Precise is ending, and as I know, Travis is looking to change default image to Trusty in Q3 2017

You can get some info here

Comment thread .travis.yml
language: node_js

node_js:
- 7
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

would you want to run tests on all LTS versions?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'm not sure that it's necessary, because we don't use node-specific features, and transpiled code should work with node from v4.

But if there are some issues with, we can run tests on all versions of course

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I usually find it pretty convenient to run tests on all LTS + latest version

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.

3 participants