coveralls - npm Package Compare versions

Comparing version 3.1.0 to 3.1.1

lib/fetchGitData.js

 'use strict';
 
-const { exec } = require('child_process');
+const { execFile } = require('child_process');
 require('./logger')();
@@ -43,3 +43,3 @@
   //-- Use git?
-  exec(`git rev-parse --verify ${git.head.id}`, err => {
+  execFile('git', ['rev-parse', '--verify', git.head.id], err => {
     if (err) {
@@ -60,3 +60,3 @@ // git is not available...
 function fetchBranch(git, cb) {
-  exec('git branch', (err, branches) => {
+  execFile('git', ['branch'], (err, branches) => {
     if (err) {
@@ -74,3 +74,3 @@ return cb(err);
 function fetchHeadDetails(git, cb) {
-  exec(`git cat-file -p ${git.head.id}`, (err, response) => {
+  execFile('git', ['cat-file', '-p', git.head.id], (err, response) => {
     if (err) {
@@ -95,3 +95,3 @@ return cb(err);
 function fetchRemotes(git, cb) {
-  exec('git remote -v', (err, remotes) => {
+  execFile('git', ['remote', '-v'], (err, remotes) => {
     if (err) {
@@ -98,0 +98,0 @@ return cb(err);

package.json

 {
   "name": "coveralls",
   "description": "takes json-cov output into stdin and POSTs to coveralls.io",
+  "version": "3.1.1",
   "keywords": [
@@ -8,16 +9,12 @@ "coverage",
   ],
-  "version": "3.1.0",
+  "author": "Gregg Caines",
+  "license": "BSD-2-Clause",
+  "repository": {
+    "type": "git",
+    "url": "git://github.com/nickmerwin/node-coveralls.git"
+  },
   "bugs": {
     "url": "https://github.com/nickmerwin/node-coveralls/issues"
   },
-  "scripts": {
-    "lint": "jshint ./lib ./test ./index.js",
-    "mocha": "_mocha -b -R spec",
-    "test-cov": "nyc npm run mocha",
-    "test-coveralls": "nyc npm run mocha && shx cat ./coverage/lcov.info | node ./bin/coveralls.js --verbose",
-    "test": "npm run lint && npm run mocha"
-  },
-  "bin": {
-    "coveralls": "./bin/coveralls.js"
-  },
+  "homepage": "https://github.com/nickmerwin/node-coveralls#readme",
   "maintainers": [
@@ -37,2 +34,16 @@ "Nick Merwin <[e-mail]> (https://coveralls.io)"
   ],
+  "bin": {
+    "coveralls": "./bin/coveralls.js"
+  },
+  "main": "index.js",
+  "directories": {
+    "test": "test"
+  },
+  "scripts": {
+    "lint": "xo",
+    "mocha": "_mocha -b -R spec",
+    "test-cov": "nyc npm run mocha",
+    "test-coveralls": "nyc npm run mocha && shx cat ./coverage/lcov.info | node ./bin/coveralls.js --verbose",
+    "test": "npm run lint && npm run mocha"
+  },
   "dependencies": {
@@ -46,3 +57,3 @@ "js-yaml": "^3.13.1",
   "devDependencies": {
-    "jshint": "^2.11.0",
+    "glob-parent": ">=5.1.2",
     "mocha": "^6.2.3",
@@ -52,3 +63,5 @@ "nyc": "^14.1.1",
     "shx": "^0.3.2",
-    "sinon": "^7.5.0"
+    "sinon": "^8.1.1",
+    "trim-newlines": ">=3.0.1",
+    "xo": "^0.24.0"
   },
@@ -58,12 +71,7 @@ "engines": {
   },
-  "main": "index.js",
-  "directories": {
-    "test": "test"
-  },
-  "repository": {
-    "type": "git",
-    "url": "git://github.com/nickmerwin/node-coveralls.git"
-  },
-  "author": "Gregg Caines",
-  "license": "BSD-2-Clause",
+  "files": [
+    "bin/coveralls.js",
+    "lib/*.js",
+    "index.js"
+  ],
   "nyc": {
@@ -75,6 +83,37 @@ "reporter": [
   },
-  "files": [
-    "{bin,lib}/*.js",
-    "index.js"
-  ]
+  "xo": {
+    "space": true,
+    "ignores": [
+      "test/fixtures/"
+    ],
+    "rules": {
+      "camelcase": "off",
+      "capitalized-comments": "off",
+      "handle-callback-err": "error",
+      "import/order": "off",
+      "no-negated-condition": "off",
+      "object-curly-spacing": [
+        "error",
+        "always"
+      ],
+      "quote-props": [
+        "error",
+        "consistent"
+      ],
+      "space-before-function-paren": [
+        "error",
+        "never"
+      ],
+      "spaced-comment": "off",
+      "unicorn/filename-case": "off"
+    },
+    "overrides": [
+      {
+        "files": "test/*.js",
+        "envs": [
+          "mocha"
+        ]
+      }
+    ]
+  }
 }

README.md

@@ -7,4 +7,14 @@ # node-coveralls
 
-Supported CI services: [Travis CI](https://travis-ci.org/), [CodeShip](https://codeship.com/), [CircleCI](https://circleci.com/), [Jenkins](https://jenkins.io/), [Gitlab CI](https://gitlab.com/), [AppVeyor](https://www.appveyor.com/), [Buildkite](https://buildkite.com/), [GitHub Actions CI](https://github.com/features/actions), [CodeFresh](https://codefresh.io)
+## Supported CI services:
 
+* [Travis CI](https://travis-ci.org/)
+* [CodeShip](https://codeship.com/)
+* [CircleCI](https://circleci.com/)
+* [Jenkins](https://jenkins.io/)
+* [Gitlab CI](https://gitlab.com/)
+* [AppVeyor](https://www.appveyor.com/)
+* [Buildkite](https://buildkite.com/)
+* [GitHub Actions CI](https://github.com/features/actions)
+* [CodeFresh](https://codefresh.io/)
+
 ## Installation:
@@ -30,3 +40,3 @@
 
-This library currently supports [Travis CI](https://travis-ci.org/) with no extra effort beyond piping the lcov output to coveralls. However, if you're using a different build system, there are a few environment variables that are necessary:
+This library currently supports [Travis CI](https://travis-ci.org/) with no extra effort beyond piping the lcov output to coveralls. However, if you're using a different build system, there are a few **necessary** environment variables:
 
@@ -39,6 +49,8 @@ - `COVERALLS_SERVICE_NAME` (the name of your build system)
 
-- `COVERALLS_SERVICE_NUMBER` (an id that uniquely identifies the build)
-- `COVERALLS_SERVICE_JOB_ID` (an id that uniquely identifies the build's job)
-- `COVERALLS_RUN_AT` (a date string for the time that the job ran. RFC 3339 dates work. This defaults to your build system's date/time if you don't set it.)
-- `COVERALLS_PARALLEL` (more info here: <https://docs.coveralls.io/parallel-build-webhook>)
+- `COVERALLS_FLAG_NAME` (a flag name to differentiate jobs, e.g. Unit, Functional, Integration)
+- `COVERALLS_SERVICE_NUMBER` (a number that uniquely identifies the build)
+- `COVERALLS_SERVICE_JOB_ID` (an ID that uniquely identifies the build's job)
+- `COVERALLS_SERVICE_JOB_NUMBER` (a number that uniquely identifies the build's job)
+- `COVERALLS_RUN_AT` (a date string for the time that the job ran. RFC 3339 dates work. This defaults to your build system's date/time if you don't set it)
+- `COVERALLS_PARALLEL` (set to `true` when running jobs in parallel, requires a completion webhook. More info here: <https://docs.coveralls.io/parallel-build-webhook>)
 
@@ -49,10 +61,14 @@ ### GitHub Actions CI
 
-If you prefer to use this package you can do it like this:
+Parallel runs example [workflow.yml](https://github.com/coverallsapp/coveralls-node-demo/blob/master/.github/workflows/workflow.yml)
 
-```yml
-env:
-  COVERALLS_REPO_TOKEN: "${{ secrets.COVERALLS_REPO_TOKEN }}"
-  COVERALLS_GIT_BRANCH: "${{ github.ref }}"
-```
+### [CircleCI Orb](https://circleci.com/)
 
+Here's our Orb for quick integration: [coveralls/coveralls](https://circleci.com/orbs/registry/orb/coveralls/coveralls)
+
+Workflow example: [config.yml](https://github.com/coverallsapp/coveralls-node-demo/blob/master/.circleci/config.yml)
+
+### [Travis-CI](https://travis-ci.org/)
+
+Parallel jobs example: [.travis.yml](https://github.com/coverallsapp/coveralls-node-demo/blob/master/.travis.yml)
+
 ### [Jest](https://jestjs.io/)
@@ -83,3 +99,3 @@
 
-Instrumenting your app for coverage is probably harder than it needs to be (read [here](http://seejohncode.com/2012/03/13/setting-up-mocha-jscoverage//)), but that's also a necessary step.
+Instrumenting your app for coverage is probably harder than it needs to be (read [here](http://seejohncode.com/2012/03/13/setting-up-mocha-jscoverage/)), but that's also a necessary step.
 
@@ -110,3 +126,3 @@ In mocha, if you've got your code instrumented for coverage, the command for a Travis CI build would look something like this:
 
-Depend on nodeunit, jscoverage and coveralls:
+Depend on nodeunit, jscoverage, and coveralls:
 
@@ -163,3 +179,3 @@ ```sh
 
-### [TAP](https://github.com/isaacs/node-tap)
+### [TAP](https://github.com/tapjs/node-tap)
 
@@ -189,5 +205,5 @@ Simply run your tap tests with the `COVERALLS_REPO_TOKEN` environment
 
-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 after all!).
+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 after all!).
 
-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.
+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.
 
@@ -194,0 +210,0 @@

Fixed alerts

No website

Quality

Package does not have a website.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

33049

increased by4.75%

Number of low quality alerts

0

decreased by-100%

Number of lines in readme file

208

increased by8.33%

Worsened metrics

Dev dependency count

8

increased by33.33%

Number of downloads

334967

decreased by-4.66%

No dependency changes