-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: remove unused deps + update knip config #3554
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This PR is published in NPM with version 0.0.0-pr-3554-20250106201303 |
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.
Did you mean to remove the profiling tool?
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.
@petertonysmith94 it should've been removed when we removed the benchmark deps I I believe.
@danielbate My understanding is that it's a separate tool to the benchmarking. |
Coverage Report:
Changed Files:
|
Summary
The original intention of this issue was to resolve any instances of false positives being detected by
knip
, given quite a few files have changes since the only instance of such was theversions.data.ts
which needed to be treated as an entryPoint since it's used in thedocs
package.The remainder of the PR removes unused dependencies across packages.
Checklist