Skip to content
This repository has been archived by the owner on Jan 12, 2020. It is now read-only.

Fix the travis build: exclude jacoco, sunjdk6; don't deploy #49

Merged
merged 3 commits into from
Jan 30, 2014

Conversation

justinsb
Copy link
Collaborator

Get the travis build working again.

Changes:

  • Create a separate maven profile for travis, without the jacoco rule (I don't think travis can find jacoco).
  • Remove sunjdk6 from the travis.yml; it is no longer available on travis so gives an error.
  • Change mvn deploy to mvn package; deployment to the snapshot repo wasn't working otherwise.

@justinsb
Copy link
Collaborator Author

@mgodave OK to merge this? Would be cool to have the Travis tick mark for all our pull requests!

@mgodave
Copy link
Owner

mgodave commented Jan 29, 2014

Yup
On Jan 29, 2014 4:47 PM, "Justin Santa Barbara" notifications@github.com
wrote:

@mgodave https://github.com/mgodave OK to merge this? Would be cool to
have the Travis tick mark for all our pull requests!

Reply to this email directly or view it on GitHubhttps://github.com//pull/49#issuecomment-33645564
.

justinsb added a commit that referenced this pull request Jan 30, 2014
Fix the travis build: exclude jacoco, sunjdk6; don't deploy
@justinsb justinsb merged commit 0365e55 into mgodave:master Jan 30, 2014
@mgodave mgodave mentioned this pull request Jan 30, 2014
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.

2 participants