Skip to content
This repository has been archived by the owner. It is now read-only.

Close #761: Integrate coveralls.io to show test coverage #820

Merged
merged 2 commits into from Oct 24, 2017

Conversation

@ollieparanoid
Copy link
Member

@ollieparanoid ollieparanoid commented Oct 24, 2017

Whether we do this or not has been discussed here, and no one spoke out against it:
#761

Preview of the README.md:
https://github.com/postmarketOS/pmbootstrap/blob/feature/coveralls/README.md

Submitting the coverage is working:
https://coveralls.io/github/postmarketOS/pmbootstrap

Oliver Smith added 2 commits Oct 24, 2017
Oliver Smith
Oliver Smith
@coveralls
Copy link

@coveralls coveralls commented Oct 24, 2017

Coverage Status

Changes Unknown when pulling 291fe39 on feature/coveralls into ** on master**.

Copy link
Member

@drebrez drebrez left a comment

Nice 👍

@ollieparanoid ollieparanoid merged commit df47b50 into master Oct 24, 2017
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls First build on master at 51.341%
Details
@ollieparanoid ollieparanoid deleted the feature/coveralls branch Oct 24, 2017
@drebrez
Copy link
Member

@drebrez drebrez commented Nov 23, 2017

@ollieparanoid branch feature/coveralls is still present, I think it can be removed (https://github.com/postmarketOS/pmbootstrap/tree/feature/coveralls)

@ollieparanoid
Copy link
Member Author

@ollieparanoid ollieparanoid commented Nov 25, 2017

Thanks for noticing, done!

PureTryOut added a commit that referenced this pull request Feb 21, 2018
* Enable coveralls.io coverage reports
* Add badge to README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.