coveralls - npm Package Compare versions

Comparing version 2.10.0 to 2.10.1

lib/convertLcovToCoveralls.js

@@ -21,3 +21,3 @@ var TRAVIS_JOB_ID = process.env.TRAVIS_JOB_ID || 'unknown';
 	var coverage = detailsToCoverage(lines.length, file.lines.details);
-	return { name     : path.relative(rootpath, path.resolve(rootpath, file.file)),
+	return { name     : path.relative(rootpath, path.resolve(rootpath, file.file)).split( path.sep ).join( "/" ),
            source   : source,
@@ -24,0 +24,0 @@ coverage : coverage	};

lib/getOptions.js

@@ -15,2 +15,4 @@ var fs = require('fs');
     options.service_job_id = process.env.TRAVIS_JOB_ID;
+    git_commit = process.env.TRAVIS_COMMIT;
+    git_branch = process.env.TRAVIS_BRANCH;
   }
@@ -77,3 +79,4 @@
     } catch(ex){
-      logger.warn("Repo token could not be determined.  Continuing without it.");
+      logger.warn("Repo token could not be determined.  Continuing without it." +
+                  "This is necessary for private repos only, so may not be an issue at all.");
     }
@@ -80,0 +83,0 @@ }

package.json

@@ -8,3 +8,3 @@ {
   ],
-  "version": "2.10.0",
+  "version": "2.10.1",
   "bugs": {
@@ -34,3 +34,3 @@ "url": "https://github.com/cainus/node-coveralls/issues"
     "lcov-parse": "0.0.6",
-    "log-driver": "1.2.1"
+    "log-driver": "1.2.4"
   },
@@ -42,3 +42,3 @@ "devDependencies": {
     "should": "1.1.0",
-    "istanbul": "0.1.45",
+    "istanbul": "0.2.10",
     "jshint": "2.1.3"
@@ -45,0 +45,0 @@ },

README.md

 #node-coveralls
-[![Build Status](https://travis-ci.org/cainus/node-coveralls.png?branch=master)](https://travis-ci.org/cainus/node-coveralls)
-[![Coverage Status](https://coveralls.io/repos/cainus/node-coveralls/badge.png?branch=master)](https://coveralls.io/r/cainus/node-coveralls?branch=master)
-[![Codeship Build Status](https://www.codeship.io/projects/de6fb440-dea9-0130-e7d9-122ca7ee39d3/status)](https://www.codeship.io/projects/5622)
-[![Build
-Status](https://drone.io/github.com/cainus/node-coveralls/status.png)](https://drone.io/github.com/cainus/node-coveralls/latest)
 
+[![Build Status][travis-image]][travis-url] [![Coverage Status][coveralls-image]][coveralls-url] [![Codeship Build Status][codeship-image]][codeship-url]
+
 [Coveralls.io](https://coveralls.io/) support for node.js.  Get the great coverage reporting of coveralls.io and add a cool coverage button ( like the one above ) to your README.
@@ -107,2 +104,18 @@
 
+[travis-image]: https://travis-ci.org/cainus/node-coveralls.svg?branch=master
+[travis-url]: https://travis-ci.org/cainus/node-coveralls
 
+[codeship-image]: https://www.codeship.io/projects/de6fb440-dea9-0130-e7d9-122ca7ee39d3/status
+[codeship-url]: https://www.codeship.io/projects/5622
+
+[coveralls-image]: https://img.shields.io/coveralls/cainus/node-coveralls.svg?branch=master
+[coveralls-url]: https://coveralls.io/r/cainus/node-coveralls?branch=master
+
+## Contributing
+
+I generally don't accept pull requests that are untested, or break the build, because I'd like to keep the quality high (this is a coverage tool afterall!).
+
+I also don't care for "soft-versioning" or "optimistic versioning" (dependencies that have ^, x, > in them, or anything other than numbers and dots).  There have been too many problems with bad semantic versioning in dependencies, and I'd rather have a solid library than a bleeding edge one.
+
+
+

test/convertLcovToCoveralls.js

@@ -72,3 +72,3 @@ var convertLcovToCoveralls = require('../index').convertLcovToCoveralls;
       return originalReadFileSync.apply(fs, arguments);
-    }
+    };
 
@@ -75,0 +75,0 @@ convertLcovToCoveralls(input, {filepath: libpath}, function(err, output){

New alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 2 instances in 1 package

Improved metrics

Number of lines in readme file

121

increased by12.04%

Worsened metrics

Total package byte prevSize

54263

decreased by-73.9%

Number of package files

23

decreased by-4.17%

Lines of code

1317

decreased by-80.88%

Number of low supply chain risk alerts

89

increased by2.3%

Number of medium supply chain risk alerts

2

increased by100%

Dependency changes

• + Addedlog-driver@1.2.4(transitive)

• - Removedlog-driver@1.2.1(transitive)

• Updatedlog-driver@1.2.4