nuxt / telemetry

Nuxt Telemetry
MIT License
205 stars 23 forks source link

perf: remove unnecessary dependencies #114

Closed antfu closed 11 months ago

antfu commented 11 months ago

From 49MB -> 41MB (the main overhead is @babel/standalone from @nuxt/kit it's shared with Nuxt tho).

https://pkg-size.dev/@nuxt%2Ftelemetry

https://github.com/nuxt/nuxt/issues/23380

codecov-commenter commented 11 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e41c397) 82.76% compared to head (69403d7) 83.11%. Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #114 +/- ## ========================================== + Coverage 82.76% 83.11% +0.35% ========================================== Files 19 19 Lines 615 616 +1 Branches 71 74 +3 ========================================== + Hits 509 512 +3 + Misses 105 102 -3 - Partials 1 2 +1 ``` | [Files](https://app.codecov.io/gh/nuxt/telemetry/pull/114?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nuxt) | Coverage Δ | | |---|---|---| | [src/consent.ts](https://app.codecov.io/gh/nuxt/telemetry/pull/114?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nuxt#diff-c3JjL2NvbnNlbnQudHM=) | `55.26% <100.00%> (-2.64%)` | :arrow_down: | | [src/utils/detect-package-manager.ts](https://app.codecov.io/gh/nuxt/telemetry/pull/114?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nuxt#diff-c3JjL3V0aWxzL2RldGVjdC1wYWNrYWdlLW1hbmFnZXIudHM=) | `100.00% <100.00%> (ø)` | | | [src/events/files.ts](https://app.codecov.io/gh/nuxt/telemetry/pull/114?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nuxt#diff-c3JjL2V2ZW50cy9maWxlcy50cw==) | `61.53% <50.00%> (+11.53%)` | :arrow_up: |

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