-
Notifications
You must be signed in to change notification settings - Fork 23
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
Codecov plugin #43
Codecov plugin #43
Conversation
057abab
to
3bad8c4
Compare
Codecov Report
@@ Coverage Diff @@
## master #43 +/- ##
=========================================
Coverage ? 87.31%
=========================================
Files ? 4
Lines ? 134
Branches ? 0
=========================================
Hits ? 117
Misses ? 17
Partials ? 0 Continue to review full report at Codecov.
|
4e849c1
to
4c78bfc
Compare
.travis.yml
Outdated
script: | ||
- make test | ||
- make dialyzer | ||
- make codecov |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this makes sense only in case of success
as the coverage report is sent only then.
Makefile
Outdated
rebar: | ||
wget https://github.com/rebar/rebar/releases/download/2.5.1/rebar &&\ | ||
chmod u+x rebar | ||
codecov: rebar3 test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test target
will be executed twice now, see https://travis-ci.org/esl/exml/jobs/426567957#L589 and https://travis-ci.org/esl/exml/jobs/426567957#L653
I think it's enough set the same dependencies for the codecov
target as for coverage-report
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some parts of the changes could be improved. See my comments to the code.
7869f5e
to
7350547
Compare
7350547
to
ccf5303
Compare
After migrating to rebar3, add codecov plugin