search

joestump / python-oauth2

A fully tested, abstract interface to creating OAuth clients and servers.
MIT License
2.99k stars 912 forks source link

Submit coverage reports to Codecov #200

Closed stevepeak closed 9 years ago

stevepeak commented 9 years ago

Steve here from Codecov. Thank you for maintaining this project! I wanted to create this PR to add our coverage reporting tool to this repository, which we use at Codecov :+1:

View a report generated from this fork

I look forward to hearing your feedback. Let me know if you have any questions!

Cheers, Steve

codecov-io commented 9 years ago

Current coverage is 96.05%

Branch #200 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found. Review entire Coverage Diff as of 08fd569

Powered by Codecov. Updated on successful CI builds.

rickhanlonii commented 9 years ago

Hey Steve,

Thanks for the PR!

Does codecov post a comment on PRs as done above or can it hook into the GitHub Status APIs? Do you have badges we could add to the README as well?

stevepeak commented 9 years ago

Hey @rickhanlonii I've added the badge in the pull request. Github comments and statuses are enabled by default. You can change their settings in the Codecov UI by clicking the "gears" icon on https://codecov.io/gh/joestump/python-oauth2. You can disable the comment if you would like by toggling off the features here.

Thank you!

jaitaiwan commented 9 years ago

I'm pretty keen! @joestump I think this would need you to enable... unless you want to transfer this one into the Open Trust like previously discussed?

stevepeak commented 9 years ago

@jaitaiwan I can enable feature if you would like. Is there any questions you have about the service? Thank you!

jaitaiwan commented 9 years ago

I'm just waiting to make sure other contribs are happy for merge.

rickhanlonii commented 9 years ago

I'm +1 on the badge and -1 on the comment so I turned off the comment. Thanks @stevepeak.

jaitaiwan commented 9 years ago

@rickhanlonii could you put this into develop?

rickhanlonii commented 9 years ago

Was thinking the same, will do.