Skip to content

Commit

Permalink
testing: replace coveralls with codecov (Submitty#51)
Browse files Browse the repository at this point in the history
* test: use codecov for reporting coverage

* update readme

* fix build error
  • Loading branch information
MasterOdin authored Sep 14, 2019
1 parent 5fb4e37 commit a7b4717
Show file tree
Hide file tree
Showing 6 changed files with 118 additions and 661 deletions.
16 changes: 16 additions & 0 deletions .codecov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
codecov:
notify:
require_ci_to_pass: true

coverage:
precision: 2
round: down

status:
project: true
patch: true
changes: false

comment:
layout: "reach,diff,flags"
behavior: once
4 changes: 2 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ script:
- npm test
- npm run build

#after_script:
# - npm run coveralls
after_script:
- npm run codecov

notifications:
email:
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# pdf-annotate.js

[![Build Status](https://travis-ci.com/Submitty/pdf-annotate.js.svg?branch=master)](https://travis-ci.com/Submitty/pdf-annotate.js)
[![Coverage Status](https://coveralls.io/repos/github/Submitty/pdf-annotate.js/badge.svg?branch=master)](https://coveralls.io/github/Submitty/pdf-annotate.js?branch=master)
[![codecov](https://codecov.io/gh/Submitty/pdf-annotate.js/branch/master/graph/badge.svg)](https://codecov.io/gh/Submitty/pdf-annotate.js)

Annotation layer for [PDF.js](https://github.com/mozilla/pdf.js).

Expand Down
5 changes: 1 addition & 4 deletions karma.conf.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,6 @@ let reporters = [
process.env.TRAVIS ? 'dots' : 'progress',
'coverage-istanbul'
];
if (process.env.COVERALLS_REPO_TOKEN) {
reporters.push('coveralls');
}

module.exports = function(config) {
config.set({
Expand Down Expand Up @@ -83,7 +80,7 @@ module.exports = function(config) {
}
},
coverageIstanbulReporter: {
reports: [ 'html', 'lcov', 'text-summary' ],
reports: [ 'html', 'lcovonly', 'text-summary' ],
dir: path.join(__dirname, 'coverage'),
combineBrowserReports: true,
fixWebpackSourcePaths: true
Expand Down
Loading

0 comments on commit a7b4717

Please sign in to comment.