Skip to content

Commit

Permalink
Merge pull request #18 from ljharb/master
Browse files Browse the repository at this point in the history
[Tests] fix coverage
  • Loading branch information
mattcg authored Aug 15, 2023
2 parents d69f938 + 9328e6d commit 479e7bb
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 5 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@ jobs:
uses: ljharb/actions/.github/workflows/node-majors.yml@main
with:
range: '>= 12'
command: npm run tests-only && npm run coverage
command: npm run tests-only
coverage: coveralls

node:
name: 'node tests'
Expand Down
5 changes: 3 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
.coveralls.yml
/.nyc_output/
/node_modules/
.nyc_output/
coverage/
node_modules/
7 changes: 7 additions & 0 deletions .nycrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"check-coverage": false,
"reporter": ["text-summary", "html", "json", "lcov"],
"exclude": [
"test"
]
}
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@
"lint": "eslint --ext=js,mjs .",
"tests-only": "nyc mocha ./test/",
"pretest": "npm run lint",
"test": "npm run tests-only",
"coverage": "nyc report --reporter=text-lcov | coveralls"
"test": "npm run tests-only"
},
"keywords": [
"iana",
Expand Down

0 comments on commit 479e7bb

Please sign in to comment.