Skip to content

Commit

Permalink
feat(deps): replace cover/istanbul with nyc (#1823)
Browse files Browse the repository at this point in the history
`istanbul` is not supported anymore, and the last version of `cover` was
released seven years ago. `cover` also depends on a vulnerable package
(which makes `npm audit` noisy). Since `make cover` was not working
anymore, replacing it with `nyc` (`istanbul`'s successor) should be
straightforward.

Signed-off-by: Matheus Marchini <mmarchini@netflix.com>
  • Loading branch information
mmarchini committed Apr 1, 2020
1 parent 1892c78 commit 361f83e
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
5 changes: 3 additions & 2 deletions .gitignore
@@ -1,5 +1,3 @@
.coverage_data
cover_html
node_modules
docs/*.html
docs/pkg
Expand All @@ -14,3 +12,6 @@ deps/javascriptlint
deps/jsstyle
package-lock.json
benchmark/results
.nyc_output/
coverage/
cover_html/
5 changes: 2 additions & 3 deletions Makefile
Expand Up @@ -21,7 +21,7 @@ ESLINT := ./node_modules/.bin/eslint
DOCUMENTATION := ./node_modules/.bin/documentation
NODEUNIT := ./node_modules/.bin/nodeunit
MOCHA := ./node_modules/.bin/mocha
NODECOVER := ./node_modules/.bin/cover
NODECOVER := ./node_modules/.bin/nyc
DOCS_BUILD := ./tools/docsBuild.js
NPM := npm
NODE := node
Expand Down Expand Up @@ -52,8 +52,7 @@ $(NODECOVER): | $(NPM_EXEC)
.PHONY: cover
cover: $(NODECOVER)
@rm -fr ./.coverage_data
$(NODECOVER) run $(NODEUNIT) ./test/*.js
$(NODECOVER) report html
$(NODECOVER) --reporter=html --reporter=text-summary --reporter=lcov $(NODEUNIT) ./test/*.js

CLEAN_FILES += $(TAP) ./node_modules/nodeunit

Expand Down
3 changes: 1 addition & 2 deletions package.json
Expand Up @@ -122,7 +122,6 @@
"autocannon": "^4.0.0",
"autocannon-compare": "^0.4.0",
"chai": "^4.2.0",
"cover": "^0.2.9",
"coveralls": "^3.0.3",
"documentation": "^11.0.0",
"eslint": "^5.16.0",
Expand All @@ -131,10 +130,10 @@
"eslint-plugin-prettier": "^3.1.0",
"glob": "^7.1.4",
"inquirer": "^3.3.0",
"istanbul": "^0.4.5",
"mkdirp": "^0.5.1",
"mocha": "^7.1.1",
"nodeunit": "^0.11.3",
"nyc": "^15.0.0",
"ora": "^1.3.0",
"pre-commit": "^1.2.2",
"prettier": "^1.17.1",
Expand Down

0 comments on commit 361f83e

Please sign in to comment.