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

Travis build fixes #244

Merged
merged 2 commits into from Jul 27, 2018

Conversation

Projects
None yet
3 participants
@ruebot
Member

ruebot commented Jul 25, 2018

What does this Pull Request do?

We ran into some TravisCI build issues on the issue-203 branch; hitting the 4MB log limit for TravisCI, and a mvn site build issue with newer versions of dependencies. I've pinned those two dependencies to specific version outlined on Stack Overflow These two commits should solve the issue.

How should this be tested?

TravisCI should turn green here, and it has already turned green here on the issue-203 branch.

Additional Notes:

When merging this, let me know before hand. I'd like to make sure the commit message is good.

@ruebot ruebot requested a review from ianmilligan1 Jul 25, 2018

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Jul 25, 2018

Codecov Report

Merging #244 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   68.71%   68.71%           
=======================================
  Files          39       39           
  Lines         911      911           
  Branches      168      168           
=======================================
  Hits          626      626           
  Misses        231      231           
  Partials       54       54

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 c73a92b...b9f356b. Read the comment docs.

codecov bot commented Jul 25, 2018

Codecov Report

Merging #244 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   68.71%   68.71%           
=======================================
  Files          39       39           
  Lines         911      911           
  Branches      168      168           
=======================================
  Hits          626      626           
  Misses        231      231           
  Partials       54       54

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 c73a92b...b9f356b. Read the comment docs.

@greebie

This comment has been minimized.

Show comment
Hide comment
@greebie

greebie Jul 26, 2018

Contributor

You are awesome @ruebot!

Contributor

greebie commented Jul 26, 2018

You are awesome @ruebot!

@ianmilligan1

LGTM, @ruebot - I'll ping you in Slack to confirm commit message.

@ianmilligan1 ianmilligan1 merged commit 290b6aa into master Jul 27, 2018

4 checks passed

codecov/patch Coverage not affected when comparing c73a92b...b9f356b
Details
codecov/project 68.71% remains the same compared to c73a92b
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@ianmilligan1 ianmilligan1 deleted the travis-build-fixes branch Jul 27, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment