Skip to content

Commit

Permalink
Merge pull request #52 from /issues/50
Browse files Browse the repository at this point in the history
Issues/50
  • Loading branch information
kopach committed Mar 24, 2020
2 parents 865f83f + 7e74dee commit 790fe21
Show file tree
Hide file tree
Showing 6 changed files with 876 additions and 136 deletions.
8 changes: 4 additions & 4 deletions .nycrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@
"extension": [".ts"],
"reporter": ["text-summary", "lcov"],
"temp-directory": "coverage/.nyc_output",
"branches": 100,
"lines": 100,
"functions": 100,
"statements": 100
"branches": 80,
"lines": 80,
"functions": 80,
"statements": 80
}
5 changes: 5 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ Plugin works with both: TypeScript (`*.ts`) and JavaScript (`*.js`) files

## ✨ Features [🔝](#-table-of-contents)

- Both [karma-coverage-istanbul-reporter](https://www.npmjs.com/package/karma-coverage-istanbul-reporter) and [karma-coverage](https://www.npmjs.com/package/karma-coverage) supported
- Both JavaScript `*.js` and TypeScript `*.ts` files support
- Multiple patterns
- Negated patterns: ['foo*', '!foobar']
Expand All @@ -67,6 +68,8 @@ npm install --save-dev karma-sabarivka-reporter

## 🔨 Usage [🔝](#-table-of-contents)

**Important Note:** if used with [karma-coverage](https://www.npmjs.com/package/karma-coverage), `'sabarivka'` should go before `'coverage'` in `reporters` list

Update `karma.conf.js`

### `include` as array of strings [🔝](#-table-of-contents)
Expand All @@ -75,6 +78,7 @@ Update `karma.conf.js`
reporters: [
// ...
'sabarivka'
// 'coverage-istanbul' or 'coverage' (reporters order is important for 'coverage' reporter)
// ...
],
coverageReporter: {
Expand All @@ -98,6 +102,7 @@ Same result may be achieved with more complex one line glob pattern
reporters: [
// ...
'sabarivka'
// 'coverage-istanbul' or 'coverage' (reporters order is important for 'coverage' reporter)
// ...
],
coverageReporter: {
Expand Down

0 comments on commit 790fe21

Please sign in to comment.