Skip to content

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

Merged
zsxwing merged 1 commit into
ReactiveX:1.xfrom
zsxwing:code-coverage
Jun 20, 2016
Merged

1.x: Publish code coverage results to codecov.io#4046
zsxwing merged 1 commit into
ReactiveX:1.xfrom
zsxwing:code-coverage

Conversation

@zsxwing
Copy link
Copy Markdown
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
Copy Markdown
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.

Comment thread build.gradle
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member

👍

@artem-zinnatullin
Copy link
Copy Markdown
Contributor

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

@zsxwing zsxwing merged commit 535fb75 into ReactiveX:1.x Jun 20, 2016
@zsxwing
Copy link
Copy Markdown
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