-
-
Notifications
You must be signed in to change notification settings - Fork 202
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
Unused exported type & interface members #910
base: main
Are you sure you want to change the base?
Conversation
commit: |
62a5eb8
to
240ae82
Compare
240ae82
to
718bf1a
Compare
This sounds like a helpful thing to warn on. At the same time, it seems perfectly valid to have unused type members for things like the result of an external REST api request. I'd type out all members of the expected response and maybe not use all of the fields at first. |
Iteration speed is definitely a thing to consider, yet could be mitigated using local JSDoc tags or config e.g. |
This PR introduces
--include typeMembers
to find unused members of exported types and interfaces.Docs: https://feat-unused-exported-type-me.knip.pages.dev/guides/type-members.
See below to install latest version through pkg.pr.new.
Dogfooded
knip --include typeMembers --fix
: 2b01993WIP. From: #848 (comment)
TODO:
--fix