Skip to content
This repository was archived by the owner on Feb 18, 2021. It is now read-only.

Update Travis CI config. #21

Closed
wants to merge 1 commit into from
Closed

Update Travis CI config. #21

wants to merge 1 commit into from

Conversation

XhmikosR
Copy link

  • specify sudo: false and fast_finish
  • remove node.js 0.11; it was an old beta and there's no point in testing against that anymore

@XhmikosR
Update Travis CI config.
54318c4 
* specify `sudo: false` and `fast_finish`
* remove node.js 0.11; it was an old beta and there's no point in testing against that anymore
@twolfson
Copy link
Contributor

I disagree with this PR. sudo: false is fine but fast_finish are for maintainers to choose. When I do development, I prefer to see all failing builds. [email protected] was out as the latest for a very long time, I think it would be negligent to remove it from Travis CI entirely.

@XhmikosR
Copy link
Author

0.11 was never stable and if people use it for production there are being ignorant to say the least.

As for fast_finish, you will see the failures when they happen. But anyway, no point in trying to convince anyone.

@XhmikosR XhmikosR closed this May 12, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@XhmikosR @twolfson