Skip to content

Conversation

kamilogorek
Copy link
Contributor

@kamilogorek kamilogorek commented Aug 3, 2018

Tested on headless Chrome and headless Firefox.

screen shot 2018-08-29 at 17 45 03

@kamilogorek kamilogorek requested a review from HazAT August 3, 2018 16:46
Copy link
Member

@HazAT HazAT left a 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.

commonjs(),
],
},
{
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated


/** JSDoc */
function serializeObject<T>(value: T, depth: number): T | string | {} {
return value;
Copy link
Member

Choose a reason for hiding this comment

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

🙈

@kamilogorek kamilogorek force-pushed the browser-integration-tests branch from 2d1a110 to 6d373bb Compare August 28, 2018 10:52
@codecov-io
Copy link

codecov-io commented Aug 28, 2018

Codecov Report

Merging #1461 into master will decrease coverage by 7.13%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            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
Impacted Files Coverage Δ
packages/utils/src/object.ts 47.82% <ø> (+0.51%) ⬆️
packages/utils/src/misc.ts 8.69% <0%> (-0.26%) ⬇️
packages/hub/src/scope.ts 95.18% <100%> (+1.13%) ⬆️
packages/raven-node/lib/client.js
packages/raven-node/lib/utils.js
packages/raven-node/index.js
...ges/raven-node/lib/instrumentation/instrumentor.js
packages/raven-node/lib/parsers.js
packages/raven-node/lib/instrumentation/http.js
packages/raven-node/lib/transports.js
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61bd254...b617683. Read the comment docs.

@kamilogorek kamilogorek force-pushed the browser-integration-tests branch 11 times, most recently from b8cc8db to 64dc00e Compare August 28, 2018 16:04
@kamilogorek kamilogorek force-pushed the browser-integration-tests branch 2 times, most recently from 24107b1 to f3e2c36 Compare August 28, 2018 16:32
@kamilogorek kamilogorek force-pushed the browser-integration-tests branch from f3e2c36 to 1ce7c60 Compare August 28, 2018 20:12
Copy link
Member

@HazAT HazAT left a 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.

@HazAT HazAT merged commit 1f3bdca into master Aug 30, 2018
@HazAT HazAT deleted the browser-integration-tests branch August 30, 2018 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants