Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Use travis trusty to speed up build #145

Closed
wants to merge 2 commits into from

Conversation

pavolloffay
Copy link
Member

No description provided.

@pavolloffay pavolloffay changed the title Use travis trusty to speed pu build Use travis trusty to speed up build Apr 22, 2017
@codecov-io
Copy link

codecov-io commented Apr 22, 2017

Codecov Report

Merging #145 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #145      +/-   ##
============================================
+ Coverage     36.22%   36.24%   +0.01%     
- Complexity      597      598       +1     
============================================
  Files            94       94              
  Lines          6349     6349              
  Branches       1052     1052              
============================================
+ Hits           2300     2301       +1     
  Misses         3866     3866              
+ Partials        183      182       -1
Impacted Files Coverage Δ Complexity Δ
.../uber/jaeger/samplers/RemoteControlledSampler.java 92.85% <0%> (+1.42%) 19% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 999283e...0ad2dd8. Read the comment docs.

@vprithvi
Copy link
Contributor

I'm not convinced that this by itself speeds up builds. Do we need to do anything additional?

build-times

@pavolloffay
Copy link
Member Author

I remember that this significantly speeded up build process in Hawkular APM. Closing.

@yurishkuro
Copy link
Member

The default precise is going away, trusty will become the default according to this. So I think it makes sense to do this, I will merge.

@yurishkuro yurishkuro reopened this Apr 26, 2017
@pavolloffay
Copy link
Member Author

closing in favour of #154

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants