karma-coveralls
Advanced tools
Comparing version 0.1.2 to 0.1.3
@@ -80,3 +80,11 @@ var fs = require('fs'); | ||
function send_to_coveralls(done, err, response, body) { | ||
if (response.statusCode >= 200 || response.statusCode < 300) { | ||
// check coveralls.io for issues, they send 200 even when down for maintenance :-\ | ||
var isJSON; | ||
try { | ||
JSON.parse(body); | ||
isJSON = true; | ||
} catch (e) { | ||
isJSON = false; | ||
} | ||
if ((response.statusCode >= 200 && response.statusCode < 300) && isJSON) { | ||
// TODO: log success sending to coveralls.io | ||
@@ -83,0 +91,0 @@ log.info("%d --- %s", response.statusCode, success(body)); |
{ | ||
"name": "karma-coveralls", | ||
"version": "0.1.2", | ||
"version": "0.1.3", | ||
"description": "A karma plugin which uploads coverage reports to coveralls.io", | ||
@@ -5,0 +5,0 @@ "main": "lib/index.js", |
@@ -22,3 +22,3 @@ #karma-coveralls | ||
coverageReporter: { | ||
type: 'lcov' // lcov or lcovonly are required for generating lcov.info files | ||
type: 'lcov', // lcov or lcovonly are required for generating lcov.info files | ||
dir: 'coverage/' | ||
@@ -40,2 +40,8 @@ } | ||
Please note that is is NOT recommended to save your repo token in plain text for everyone to see. | ||
Treat it like a password. If you need to include it in a public repo you should [encrypt it](http://docs.travis-ci.com/user/build-configuration/#Secure-environment-variables). | ||
Be patiant when sending coverage information to coveralls, it can take upto 4 hours for things to | ||
start showing up properly. | ||
##Contribution | ||
@@ -42,0 +48,0 @@ |
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
7262
95
61