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

Added token for codecov.io #21

Merged
merged 2 commits into from
Jun 6, 2016
Merged

Conversation

rafaparadela
Copy link

This PR just adds the token in the command to generate reports in Codecov.io

@fedefernandez @juanpedromoreno Could you take a look please? Thank you.

@codecov-io
Copy link

codecov-io commented Jun 6, 2016

Current coverage is 0.68%

Merging #21 into master will increase coverage by 0.68%

@@            master       #21   diff @@
========================================
  Files           14        14          
  Lines          147       147          
  Methods          0         0          
  Messages         0         0          
  Branches         1         1          
========================================
+ Hits             0         1     +1   
+ Misses         147       146     -1   
  Partials         0         0          

Powered by Codecov. Last updated by 6cc84e9...40c87cd

@rafaparadela
Copy link
Author

It seems like if reports are been submitted properly now:

https://codecov.io/gh/47deg/github4s/branch/rafa-fix-codecov-reports

@fedefernandez @juanpedromoreno Is good to merge? Thank you

@juanpedromoreno
Copy link
Member

LGTM!

@rafaparadela rafaparadela merged commit f9527ee into master Jun 6, 2016
@rafaparadela rafaparadela deleted the rafa-fix-codecov-reports branch June 6, 2016 13:59
@rafaparadela rafaparadela mentioned this pull request Jun 6, 2016
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.

3 participants