Skip to content

chore(travis): Test against latest Node LTS versions#143

Merged
timche merged 3 commits intoredux-utilities:masterfrom
JaKXz:chore/travis-config-update
Oct 6, 2016
Merged

chore(travis): Test against latest Node LTS versions#143
timche merged 3 commits intoredux-utilities:masterfrom
JaKXz:chore/travis-config-update

Conversation

@JaKXz
Copy link
Copy Markdown
Contributor

@JaKXz JaKXz commented Oct 6, 2016

Since Node 5 is past its useful life, speed up the build process by just
specifying LTS versions 4 and 6.

Since Node 5 is past its useful life, speed up the build process by just
specifying LTS versions 4 and 6.
Comment thread .travis.yml Outdated
@@ -1,8 +1,9 @@
sudo: false
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.

sudo is by default false for repos since 2015 on Travis, also see log.

Comment thread .travis.yml Outdated
- "6"
- "5"
- "4"

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.

Any reasons for this whitespace? If not, I'd suggest to remove it as well to stay on the context.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just a habit to give the code some space. I'll remove it.

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.

No problem, thanks ;)

@timche timche merged commit 1e48e1c into redux-utilities:master Oct 6, 2016
@JaKXz JaKXz deleted the chore/travis-config-update branch October 6, 2016 21:23
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.

2 participants