Skip to content

CI config changes to speed up tests#260

Merged
rwgdrummer merged 1 commit intomasterfrom
GEOWAVE-257
Mar 13, 2015
Merged

CI config changes to speed up tests#260
rwgdrummer merged 1 commit intomasterfrom
GEOWAVE-257

Conversation

@chrisbennight
Copy link
Copy Markdown
Contributor

Updated accumulo 1.5.x version to 1.5.2

@chrisbennight
Copy link
Copy Markdown
Contributor Author

Hmm, not sure why this can't be merged automatically - I rebased against master. Might me something with the formatting cleanup of the Pom. Since that is hard to read, changes were

  • increment default versions of
    • geotools
    • geoserver
    • accumulo
    • hadoop
    • jacoco
  • move hortonworks and cloudera profiles over to their respective versions/builds of accumulo
  • travis.yml changes
    • make document generation only run on master
    • forked coveralls plugin tests first to see if it is in the local maven cache before clone and install
    • document generation phase moved to before install so it gets included in the travis repo cache
  • reduce build matrix to 4 permutations of versions

Comment thread pom.xml Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this '-'? Can I remove it. If so, I have the rebased version ready to push and then merge.

@rwgdrummer
Copy link
Copy Markdown
Contributor

See my comment in the diff. I see the changes. I am ready to purge, but I need to verify the one comment.

@rwgdrummer
Copy link
Copy Markdown
Contributor

See my comment in the diff.
Please verify so I can complete the merge.

@chrisbennight
Copy link
Copy Markdown
Contributor Author

Nope, don't need the dash- not sure where it creeped in

rwgdrummer added a commit that referenced this pull request Mar 13, 2015
CI config changes to speed up tests
@rwgdrummer rwgdrummer merged commit 92f6a25 into master Mar 13, 2015
@rwgdrummer rwgdrummer deleted the GEOWAVE-257 branch March 13, 2015 22:00
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.

2 participants