Skip to content

Conversation

@shoota
Copy link
Contributor

@shoota shoota commented Jul 10, 2018

TL;DR

Closes #612

Check this pr.

How to check

  • check CI

Check list

  • passed CI

@ghost ghost assigned shoota Jul 10, 2018
@ghost ghost added the in progress label Jul 10, 2018
"build": "tsc",
"clean": "rimraf coverage lib docs",
"prepublish": "npm run build",
"test": "karma start --single-run"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

karma test will run from root packages. Test files are defined packages/*/test/**/*.ts.
https://github.com/userdive/agent.js/blob/master/karma.conf.ts#L103

@codecov
Copy link

codecov bot commented Jul 10, 2018

Codecov Report

Merging #735 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #735   +/-   ##
=======================================
  Coverage   91.38%   91.38%           
=======================================
  Files          23       23           
  Lines         592      592           
  Branches      103      103           
=======================================
  Hits          541      541           
  Misses         27       27           
  Partials       24       24

@shoota shoota requested a review from dkimura July 11, 2018 01:04
@shoota shoota added QA and removed in progress labels Jul 11, 2018
@shoota shoota requested a review from heavenshell July 11, 2018 01:05
Copy link

@dkimura dkimura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dkimura dkimura merged commit bb91c75 into master Jul 11, 2018
@dkimura dkimura deleted the fix/karma-fixture branch July 11, 2018 01:55
@ghost ghost removed the QA label Jul 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants