Skip to content
This repository was archived by the owner on Mar 26, 2021. It is now read-only.

Updated README, added gulp-coveralls example #45

Merged
merged 1 commit into from Dec 13, 2015
Merged

Updated README, added gulp-coveralls example #45

merged 1 commit into from Dec 13, 2015

Conversation

ghost
Copy link

@ghost ghost commented Dec 8, 2015

Two small documentation changes:

  • Changed the file paths in the examples to be less confusing - gulp.src() takes the test files, cover.instrument() takes the source files being covered. The paths used in the examples seem to imply the opposite. It's obvious enough when adding coverage to an existing test runner task, but 0% line coverage, 0% statement coverage, 0% block coverage, 0 SLOC...? #32 suggests I'm not the first to be momentarily confused..
  • Added an example for usage with gulp-coveralls. I've since seen there's a good example in the gulpfile.js, but the readme suggests it's not even possible (says json and html are the only supported formats). I only realised how simple it is after seeing Emit raw LCOV too #25 and since I'd guess it's a fairly common use case, maybe deserves its own example?

dylanb added a commit that referenced this pull request Dec 13, 2015
Updated README, added gulp-coveralls example
@dylanb dylanb merged commit 08bea7b into dylanb:master Dec 13, 2015
@dylanb
Copy link
Owner

dylanb commented Dec 13, 2015

Thanks for your contribution

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant