-
-
Notifications
You must be signed in to change notification settings - Fork 222
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
Vite virtual imports not recognized by Knip #370
Comments
How can we translate |
I am not saying we necessarily need to translate it, but maybe knip should not mark these as unused since it is very confusing for users. Either we should ignore them by default when using vite or at least make it clear when using the vite plugin (in the docs perhaps). |
🚀 This issue has been resolved in v3.3.3. See Release 3.3.3 for release notes. |
Knip no longer sanitizes Another option could be to ignore them altogether, but that would be harder to implement and it may lead to confusion (not sure as I know virtually nothing about this technique). |
When using vite virual imports "virtual" appears as an unused dependency when running knip.
Example:
Knip output:
Actual package name:
vite-plugin-pwa
The text was updated successfully, but these errors were encountered: