-
-
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
Issues are not reported in monorepo when workspaces are setup in knip.json #722
Comments
If yup is listed in both root and shared package, seems it's unused in root. Expected behavior? |
Expected behavior is all issues from all packages are reported. It does not matter what dependency, now replaced |
Sorry, maybe checking that from phone late at night wasn't the best idea. Non-glob additional workspace names were indeed not added properly (e.g. just Thanks for the report! |
🚀 This issue has been resolved in v5.25.2. See Release 5.25.2 for release notes. Using Knip in a commercial project? Please consider becoming a sponsor. |
Interesting that I had an issue with the glob workspace name in |
Thanks @vubogovich! |
Our monorepo consists of independent packages and does not use npm workspaces yet. As a result, it and requires
npm install
for each of the package separately. We have setup prettier and eslint in the root package and typescript is configurated independently.knip
is installed on the root package andknip.json
has the following content:However,
knip
reports issues only in the root package.knip --workspace packages/shared
suggested in the docs didn't help.The issues were unused dependency and file
unused.ts
. Very nice template for reproductions 👍 here is my version.Thanks a lot for very useful tool! I used it in the normal repo and enjoyed a lot ❤️
The text was updated successfully, but these errors were encountered: