Skip to content

Conversation

@addisonElliott
Copy link
Contributor

The coverage shield was showing unknown and clicking on the link would take to a page not indicating the current coverage of the SDK.

Change the shield to use codecov (like the Parse-SDK-iOS-OSX Github repo does) to report the code coverage.

The coverage shield was showing unknown and clicking on the link would take to a page not indicating the current coverage of the SDK.

Change the shield to use codecov (like the Parse-SDK-iOS-OSX Github repo does) to report the code coverage.
@codecov
Copy link

codecov bot commented Aug 17, 2017

Codecov Report

Merging #708 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #708   +/-   ##
=========================================
  Coverage     53.06%   53.06%           
  Complexity     1732     1732           
=========================================
  Files           132      132           
  Lines         10254    10254           
  Branches       1432     1432           
=========================================
  Hits           5441     5441           
  Misses         4364     4364           
  Partials        449      449

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa2c620...c9d6642. Read the comment docs.

@rogerhu
Copy link
Contributor

rogerhu commented Aug 17, 2017

Resolve =)

@addisonElliott
Copy link
Contributor Author

Good to go here

👍

@Jawnnypoo Jawnnypoo self-requested a review August 18, 2017 22:30
@Jawnnypoo Jawnnypoo merged commit 2825459 into parse-community:master Aug 18, 2017
@Jawnnypoo
Copy link
Member

Thanks! Sorry, didn't mean to remove the other reviewers 😛

@addisonElliott addisonElliott deleted the update-coverage-shield branch November 24, 2017 14:42
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