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

Testing is done at travis-ci, packaging on circleci #1169

Merged
merged 7 commits into from
Apr 12, 2016
Merged

Conversation

simonharrer
Copy link
Contributor

@simonharrer simonharrer added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers build-system labels Apr 11, 2016
@matthiasgeiger
Copy link
Member

There seems to be a flaw in your description...

Coveralls and Coverity are removed and replaced by codecov and codacy?

@koppor
Copy link
Member

koppor commented Apr 12, 2016

Reasoning:

  • Most coverity have been fixed.
    • 13 identified medium issues are false positives.
    • we needed gradle work arounds to get it working
  • Codacy invests in its future and will integrate other tooling; also offers a badge
  • Coveralls is slower as codecov; used

@simonharrer simonharrer merged commit d9aa2c1 into master Apr 12, 2016
@stefan-kolb stefan-kolb deleted the ci-changes branch April 13, 2016 18:03
@stefan-kolb stefan-kolb restored the ci-changes branch April 13, 2016 18:03
@stefan-kolb stefan-kolb deleted the ci-changes branch April 13, 2016 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build-system status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants