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

Cut out a bunch of Travis builds to see if things are better. #503

Merged
merged 1 commit into from
Aug 8, 2019

Conversation

non
Copy link
Contributor

@non non commented Aug 8, 2019

It's possible this is too aggressive (or not aggressive enough) but I
wanted to see how much faster and more reliable things would be with
these settings.

It's possible this is too aggressive (or not aggressive enough) but I
wanted to see how much faster and more reliable things would be with
these settings.
@non non mentioned this pull request Aug 8, 2019
@non
Copy link
Contributor Author

non commented Aug 8, 2019

Well, things are still super flaky (and it takes awhile to get scheduled). On the up-side, we only had 14 jobs and the longest took only 8 minutes, which is an improvement.

@non
Copy link
Contributor Author

non commented Aug 8, 2019

@ashawley What do you think? It might be nice to do something like this (at least until Travis gets less flaky). For the last couple days I think we've all had to baby-sit these Travis builds and click restart on the failed jobs which is annoying.

@ashawley
Copy link
Contributor

ashawley commented Aug 8, 2019

Yeah, makes sense. Looks like it cuts the build times in half.

@non non merged commit ce06c01 into typelevel:master Aug 8, 2019
@non non deleted the topic/less-travis branch August 8, 2019 20:57
@SethTisue
Copy link
Member

SethTisue commented Aug 8, 2019

I don't know the history of SBT_PARALLEL. would it make sense to use include: to add a single job with SBT_PARALLEL=false?

and if that doesn't make sense, then I wonder if it might make sense to eliminate SBT_PARALLEL entirely

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