vercel / speed-insights

Vercel Speed Insights package
https://vercel.com/docs/speed-insights
Apache License 2.0
55 stars 6 forks source link

feat: shorter framework names #18

Closed feugy closed 7 months ago

feugy commented 7 months ago

๐Ÿ““ What's in there?

When reporting vitals and metrics, we should use a shorter framework name to avoid very long values like @vercel/speed-insights/sveltekit

A shorter prefix like vercel should be enough, and will give vercel/next, vercel/sveltekit or vercel/nuxt

๐Ÿงช How to test?

Have a try at one of our deployments, with dev tools open on network, filtering for vitals

image
vercel[bot] commented 7 months ago

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Comments Updated (UTC)
speed-insights-nextjs โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Nov 28, 2023 2:07pm
speed-insights-nuxt โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Nov 28, 2023 2:07pm
speed-insights-remix โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Nov 28, 2023 2:07pm
speed-insights-sveltekit โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Nov 28, 2023 2:07pm
speed-insights-vue โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Nov 28, 2023 2:07pm
linear[bot] commented 7 months ago
ALY-1087 A shorten framework name for @vercel/speed-insights

When reporting vitals and metrics, we should use a shorter framework name to avoid very long values like `vercel/speed-insights/sveltekit` A shorter prefix like `vercel` should be enough, and will give `vercel/next`, `vercel/sveltekit` or `vercel/nuxt`

feugy commented 7 months ago

We concluded that we can afford these long names. At least they are explicit!