Skip to content

Commit

Permalink
travis.yml: simplify piping coverage to coveralls
Browse files Browse the repository at this point in the history
`tap` automatically uploads coverage to coveralls for branches, but not PR's.

That required an ugly workaround in .travis.yml to manually upload only for PR's.

Since we now use `nyc` directly, and pass the the `--no-cov` flag in our `tap` commands, this is no longer a problem.
  • Loading branch information
jamestalmage committed Dec 22, 2015
1 parent 7fce77b commit 2ba0686
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@ node_js:
- '0.12'
- '0.10'
after_success:
- '[ -z "$COVERALLS_REPO_TOKEN" ] && tap --coverage-report=text-lcov | ./node_modules/.bin/coveralls'
- 'cat ./coverage/lcov.info | ./node_modules/.bin/coveralls'

0 comments on commit 2ba0686

Please sign in to comment.