search

newrelic / newrelic-browser-agent

New Relic Browser Agent
Apache License 2.0
77 stars 40 forks source link

chore: Logging optimizations, must wrap in array #1071

Closed metal-messiah closed 3 months ago

metal-messiah commented 3 months ago

Follow up to steel thread issues

Overview

Related Issue(s)

Testing

github-actions[bot] commented 3 months ago

Asset Size Report

Merging this pull request will result in the following asset size changes:

Agent Asset Previous Size New Size Diff
lite loader 34.59 kB / 12.05 kB (gzip) 34.59 kB / 12.05 kB (gzip) 0% / 0% (gzip)
lite async-chunk 49.95 kB / 16.13 kB (gzip) 49.95 kB / 16.13 kB (gzip) 0% / 0% (gzip)
pro loader 55.24 kB / 18.66 kB (gzip) 55.24 kB / 18.65 kB (gzip) 0% / -0.01% (gzip)
pro async-chunk 95.87 kB / 28.82 kB (gzip) 95.88 kB / 28.82 kB (gzip) 0% / 0.01% (gzip)
spa loader 63.39 kB / 21.14 kB (gzip) 63.39 kB / 21.14 kB (gzip) 0% / 0% (gzip)
spa async-chunk 111.14 kB / 33.25 kB (gzip) 111.14 kB / 33.26 kB (gzip) 0% / 0.01% (gzip)
lite-polyfills loader 128.55 kB / 41.37 kB (gzip) 128.55 kB / 41.37 kB (gzip) 0% / 0% (gzip)
lite-polyfills async-chunk 64.69 kB / 18.46 kB (gzip) 64.69 kB / 18.46 kB (gzip) 0% / 0% (gzip)
pro-polyfills loader 150.96 kB / 47.64 kB (gzip) 150.96 kB / 47.65 kB (gzip) 0% / 0.01% (gzip)
pro-polyfills async-chunk 118.12 kB / 31.2 kB (gzip) 118.13 kB / 31.2 kB (gzip) 0% / 0.01% (gzip)
spa-polyfills loader 159.21 kB / 49.79 kB (gzip) 159.21 kB / 49.79 kB (gzip) 0% / 0% (gzip)
spa-polyfills async-chunk 135.12 kB / 35.83 kB (gzip) 135.13 kB / 35.83 kB (gzip) 0% / 0.01% (gzip)
codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.71%. Comparing base (9a9fad5) to head (fe6229f).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1071 +/- ## ======================================= Coverage 87.71% 87.71% ======================================= Files 162 162 Lines 7234 7234 Branches 1415 1415 ======================================= Hits 6345 6345 Misses 773 773 Partials 116 116 ``` | [Flag](https://app.codecov.io/gh/newrelic/newrelic-browser-agent/pull/1071/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=newrelic) | Coverage Δ | | |---|---|---| | [unit-tests](https://app.codecov.io/gh/newrelic/newrelic-browser-agent/pull/1071/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=newrelic) | `76.44% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=newrelic#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.