/snapweb

add coverage reporting #17

Merged
merged 3 commits into from Jun 21, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@earnubs @coveralls @elopio @stevenwilkin
@earnubs
Contributor

earnubs commented Jun 20, 2016

No description provided.

Stephen Stewart added some commits Jun 20, 2016

add coverage reporting
158f844
revert to error reporting

Some checks were not successful

1 failing and 1 successful checks
continuous-integration/travis-ci/pr — The Travis CI build failed
Details
coverage/coveralls — Coverage remained the same at 50.325%
Details
a532862
@coveralls

coveralls commented Jun 20, 2016 •
Edited 1 time

Coverage Status

Coverage remained the same at 50.325% when pulling a532862 on earnubs:coverage into 7c5fb86 on snapcore:master.
@coveralls

coveralls commented Jun 20, 2016 •
Edited 1 time

Coverage Status

Coverage remained the same at 50.325% when pulling a532862 on earnubs:coverage into 7c5fb86 on snapcore:master.
oops, include istanbul

All checks have passed

2 successful checks
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+5.4%) to 55.769%
Details
392bad2
@coveralls

coveralls commented Jun 20, 2016 •
Edited 1 time

Coverage Status

Coverage increased (+5.4%) to 55.769% when pulling 392bad2 on earnubs:coverage into 7c5fb86 on snapcore:master.
@elopio
Member

elopio commented Jun 20, 2016

👍 this looks great, and even coveralls is making the right thing there.
Thanks! I'm going to close mine, and leave this one here for @stevenwilkin to look at it.

@elopio elopio referenced this pull request Jun 20, 2016

Closed

Add karma-coverage #16

@stevenwilkin
Member

stevenwilkin commented Jun 21, 2016

🚢

@stevenwilkin stevenwilkin merged commit 2e530d4 into snapcore:master Jun 21, 2016
2 checks passed

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+5.4%) to 55.769%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment