Skip to content

1.x: Publish code coverage results to codecov.io #4046

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

Merged
merged 1 commit into from
Jun 20, 2016

Conversation

zsxwing
Copy link
Member

@zsxwing zsxwing commented Jun 20, 2016

This is an example built in my repo: https://codecov.io/gh/zsxwing/RxJava/

@zsxwing zsxwing added the Build label Jun 20, 2016
@zsxwing zsxwing self-assigned this Jun 20, 2016
@artem-zinnatullin
Copy link
Contributor

Yes! I actually started the same in my fork previous week 😸.

@akarnokd: 2 most important things that codecov gives:

  1. Code coverage for each PR and configurable commit status check, it'll report that some changes in files from PR are not covered and how overall coverage of the project was affected by PR.
  2. At any time you can check coverage of particular file, take a look https://codecov.io/gh/zsxwing/RxJava/tree/1.x/src/main/java/rx .

We use it in several projects: StorIO, qualitymatters and others, it makes testing more fun and improves quality of the project.

@@ -6,6 +6,7 @@ buildscript {
description = 'RxJava: Reactive Extensions for the JVM – a library for composing asynchronous and event-based programs using observable sequences for the Java VM.'

apply plugin: 'java'
apply plugin: 'jacoco'
Copy link
Contributor

Choose a reason for hiding this comment

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

Please specify version of jacoco, like this:

jacoco {
  toolVersion = '0.7.7.201606060606' // See http://www.eclemma.org/jacoco/.
}

Copy link
Member Author

Choose a reason for hiding this comment

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

Added.

@codecov-io
Copy link

Current coverage is 80.32%

No coverage report found for 1.x at b8bccd9.

Powered by Codecov. Last updated by b8bccd9...af876b9

@akarnokd
Copy link
Member

👍

@artem-zinnatullin
Copy link
Contributor

👍 yay, so happy about it, thx @zsxwing!

@zsxwing zsxwing merged commit 535fb75 into ReactiveX:1.x Jun 20, 2016
@zsxwing
Copy link
Member Author

zsxwing commented Jun 20, 2016

PR for 2.x is on the way :)

@zsxwing zsxwing deleted the code-coverage branch June 20, 2016 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants