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

Split build tasks on travis fixes #1242 #1256

Merged
merged 1 commit into from
Aug 3, 2016
Merged

Conversation

kailuowang
Copy link
Contributor

@kailuowang kailuowang commented Aug 2, 2016

I believe the frequent scalajs build failure is caused by out of memory, So splitting the tasks helped. This is verified by #1254 which passes two builds consecutively.

Update: thanks to @mikejcurry in his comment here I understand we are at the upper bound of memory available on travis, i.e. we can't increase jvm memory to solve this problem.

I believe the frequent scalajs build failure is caused by out of memory, So splitting the tasks helped. This is verified by #1254 which passes two builds consecutively.
@codecov-io
Copy link

Current coverage is 90.13% (diff: 100%)

Merging #1256 into master will not change coverage

@@             master      #1256   diff @@
==========================================
  Files           243        243          
  Lines          3285       3285          
  Methods        3231       3231          
  Messages          0          0          
  Branches         51         51          
==========================================
  Hits           2961       2961          
  Misses          324        324          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 0c5f48c...6565ff6

@ceedubs
Copy link
Contributor

ceedubs commented Aug 3, 2016

👍 LGTM. It will be so nice to have builds working reliably again.

@travisbrown
Copy link
Contributor

👍 (and fingers crossed that the successes you've seen so far aren't just a fluke).

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.

4 participants