-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Browser integration tests #1461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only one minor thing, besides that the tests are failing.
packages/browser/rollup.config.js
Outdated
@@ -30,6 +30,32 @@ export default [ | |||
commonjs(), | |||
], | |||
}, | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could de-dupe it by having this object top level, and for minified we inject uglify
into plugins.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
@@ -202,8 +202,6 @@ function serializeValue<T>(value: T): T | string { | |||
|
|||
/** JSDoc */ | |||
function serializeObject<T>(value: T, depth: number): T | string | {} { | |||
return value; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙈
2d1a110
to
6d373bb
Compare
Codecov Report
@@ Coverage Diff @@
## master #1461 +/- ##
==========================================
- Coverage 79.68% 72.55% -7.14%
==========================================
Files 51 41 -10
Lines 2063 1246 -817
Branches 474 238 -236
==========================================
- Hits 1644 904 -740
+ Misses 409 333 -76
+ Partials 10 9 -1
Continue to review full report at Codecov.
|
b8cc8db
to
64dc00e
Compare
24107b1
to
f3e2c36
Compare
f3e2c36
to
1ce7c60
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is super nice,
one more thing since we are changed quite a bit in travis now.
I wrote travis that we want to move to travis-ci.com to have a even better github integration.
Tested on headless Chrome and headless Firefox.