[Pkg-javascript-commits] [node-coveralls] 14/332: fix README. fix off-by-one error. version bump 2.0.4.

[Bastien Roucariès]

This is an automated email from the git hooks/post-receive script.

rouca pushed a commit to branch master
in repository node-coveralls.

commit 67effe59fbaefa65e69f5326c8bcd867537aea00
Author: cainus <gregg at caines.ca>
Date:   Thu Mar 28 01:01:06 2013 -0700

    fix README.  fix off-by-one error. version bump 2.0.4.
---
 README.md                     | 6 +++---
 lib/convertLcovToCoveralls.js | 2 +-
 package.json                  | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/README.md b/README.md
index b785070..9a79812 100644
--- a/README.md
+++ b/README.md
@@ -4,11 +4,11 @@
 
 Installation: Add the latest version of `coveralls` to your package.json.
 
-This script ( `bin/coveralls.js` ) can take standard input in the JSON format from [mocha](http://visionmedia.github.com/mocha/)'s JSONCov reporter and send it to coveralls.io to report your code coverage there.  It needs to run from [travis-ci](http://about.travis-ci.org/docs/user/languages/javascript-with-nodejs/) to work.
+This script ( `bin/coveralls.js` ) can take standard input from any tool that emits the lcov data format (including [mocha](http://visionmedia.github.com/mocha/)'s [LCov reporter](https://npmjs.org/package/mocha-lcov-reporter)) and send it to coveralls.io to report your code coverage there.  It needs to run from [travis-ci](http://about.travis-ci.org/docs/user/languages/javascript-with-nodejs/) to work.
 
 Instrumenting your app for coverage is probably harder than it needs to be (read [here](http://www.seejohncode.com/2012/03/13/setting-up-mocha-jscoverage/) or [here](http://tjholowaychuk.com/post/18175682663/mocha-test-coverage)), but that's also a necessary step. 
 
-Once your app is instrumented for coverage, and building in travis-ci, you just need to set your mocha reporter to the JSONCov reporter, and pipe the output to `./node_modules/coveralls/bin/coveralls.js`.  
+Once your app is instrumented for coverage, and building in travis-ci, you just need to pipe the lcov output to `./node_modules/coveralls/bin/coveralls.js`.
 
-Check out an example [Makefile](https://github.com/cainus/Prozess/blob/master/Makefile) from one of my projects for an example, especially the test-coveralls build target.  Note: Travis runs `npm test`, so whatever target you create in your Makefile must be the target that `npm test` runs.  Also note that the TRAVIS_JOB_ID is necessary.
+Check out an example [Makefile](https://github.com/cainus/urlgrey/blob/master/Makefile) from one of my projects for an example, especially the test-coveralls build target.  Note: Travis runs `npm test`, so whatever target you create in your Makefile must be the target that `npm test` runs.  Also note that the TRAVIS_JOB_ID is necessary.
 
diff --git a/lib/convertLcovToCoveralls.js b/lib/convertLcovToCoveralls.js
index c390aab..14d637e 100644
--- a/lib/convertLcovToCoveralls.js
+++ b/lib/convertLcovToCoveralls.js
@@ -6,7 +6,7 @@ var path = require('path');
 var detailsToCoverage = function(length, details){
   var coverage = new Array(length);
   details.forEach(function(obj){
-    coverage[obj.line] = obj.hit;
+    coverage[obj.line - 1] = obj.hit;
   });
   return coverage;
 };
diff --git a/package.json b/package.json
index 2cb0013..19245cd 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,7 @@
       "name": "coveralls",
       "description" : "takes json-cov output into stdin and POSTs to coveralls.io",
       "keywords" : ["coverage", "coveralls"], 
-      "version": "2.0.3",
+      "version": "2.0.4",
       "bugs": {
        "url": "https://github.com/cainus/node-coveralls/issues"
       },

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/node-coveralls.git