Lars Kappert
Lars Kappert
The only thing I want to add at this point as that one should definitely not wait >10 minutes for any run, especially when iterating on the config. There is...
Oh `ignoreExportsUsedInFile` slows things down too (you mentioned looking for unused deps only anyway).
No, respecting `.npmignore` wouldn't help. Just closed that ticket with some reasoning.
Yes and no. You're technically correct, but from a user's perspective usually only `dependencies` is relevant: - https://knip.dev/features/rules-and-filters - https://knip.dev/reference/cli#--dependencies The idea is that `--dependencies` is a shorthand flag and...
Thanks for bringing this up again, and the efforts already put in. I'm going to change the title to make it clear that we should really only merge if we...
Knip requires at least Node.js v18, tinyglobby v12. And I don't expect Knip to reach the `fs.glob` threshold too soon, as it might be helpful in legacy projects trying to...
Nice work, thanks!
:rocket: _This pull request is included in v5.81.0. See [Release 5.81.0](https://github.com/webpro-nl/knip/releases/tag/knip%405.81.0) for release notes._ _Using Knip in a commercial project? Please consider [becoming a sponsor](https://knip.dev/sponsors)._
Currently, Knip doesn't even read the unused files at all. Semantically, not sure if your suggestion results in clear DX. Maybe some annotation in the head of the file so...
The file isn't necessarily meant to be _public_, maybe it should just be excluded (temporarily) from the report for whatever reason.