Lars Kappert comments

Results 640 comments of


                                            Lars Kappert

🧩 Complex plugin challenge

> > users need to be aware that overrides/differences here need to be mirrored in their Knip config > > That's what I'm trying to avoid Yes, I gathered the...

🐛 A custom stylelint plugin causes knip crash

Thanks for the report, Zhang! The repro url returns a 404, though.

🐛 A custom stylelint plugin causes knip crash

Looks like we need to check whether the value is a string before using it: https://github.com/webpro-nl/knip/blob/main/packages/knip/src/plugins/stylelint/index.ts#L19 Someone up for a PR maybe? :)

💡 Pass in configuration as command-line argument

This has come up before (in #823, also #573) and tbh the goal still isn't to have parity between CLI arguments and configuration file. Yet where to draw the line?...

💡 Pass in configuration as command-line argument

That's not a bad idea, but doesn't strike me as a common thing across tooling. Just an idea: it isn't that many more characters to do something create some sort...

💡 Pass in configuration as command-line argument

Another option is something like `knip -c ../knip.json` (https://knip.dev/reference/cli#--config-file)

🐛 warning from ladle plugin

I'm not familiar with this issue. How to reproduce this?

🐛 warning from ladle plugin

Closing due to inactivity. Feel free to open a new ticket if there's still interest, preferably with [an issue reproduction](https://knip.dev/guides/issue-reproduction) and/or a pull request.

Add `@tanstack/react-router` plugin

Thanks for the PR, @nickrttn! > This adds support for the file-based routing Vite/RsPack/RsBuild/Webpack plugin of `@tanstack/react-router`. The only reliable way I saw to parse the config files of those...