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

Disable a few travis builds to speed us up #622

Merged
merged 2 commits into from
Feb 14, 2019

Conversation

scravy
Copy link
Member

@scravy scravy commented Feb 14, 2019

Disables two travis builds and runs them in the daily cron job (at night) only.

  • The windows 64 bit one (win32 bit should be sufficiently similar to the win64 bit build and sufficiently different form our 64-bit linux/unix machines).
  • One of the builds which run functional tests. There's still a 32-bit and a 64-bit build running functional tests on linux.

Here's the documentation on conditional builds in travis:

And here's the documentation on the available conditions:

This is to tackle our long travis queue.

Signed-off-by: Julian Fleischer julian@thirdhash.com

@scravy scravy added the ci Continuous integration label Feb 14, 2019
@scravy scravy added this to the 0.1 milestone Feb 14, 2019
@scravy scravy requested review from thothd and a team February 14, 2019 14:55
Signed-off-by: Julian Fleischer <julian@thirdhash.com>
Signed-off-by: Julian Fleischer <julian@thirdhash.com>
@scravy scravy force-pushed the reduce-number-of-travis-builds branch from 9869778 to 19fb12e Compare February 14, 2019 15:01
@cornelius cornelius changed the title Disable a few traivs builds to speed us up Disable a few travis builds to speed us up Feb 14, 2019
@cornelius
Copy link
Member

I like the idea. Concept ACK.

@scravy
Copy link
Member Author

scravy commented Feb 14, 2019

@cornelius I implemented the only-run-in-cron-job feature too, which also makes it just a two line change.

Copy link

@castarco castarco left a comment

Choose a reason for hiding this comment

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

utACK 19fb12e , nice to save some build time.

@cornelius
Copy link
Member

@cornelius I implemented the only-run-in-cron-job feature too, which also makes it just a two line change.

Yeah, I saw that. That's really nice.

@scravy scravy merged commit 97d260e into dtr-org:master Feb 14, 2019
@scravy scravy deleted the reduce-number-of-travis-builds branch February 18, 2019 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants