Lars Kappert comments

Results 640 comments of


                                            Lars Kappert

disable rule for workspace

Good idea, leaving this ticket open until it's done

Respect up-the-tree `.gitignore` files

Since I'm convinced this should be in Knip I've worked on this feature today. Would appreciate your feedback and verify if it works as expected. Performance should not be affected...

Respect up-the-tree `.gitignore` files

:rocket: _This issue has been resolved in v5.30.0. See [Release 5.30.0](https://github.com/webpro-nl/knip/releases/tag/5.30.0) for release notes._ _Using Knip in a commercial project? Please consider [becoming a sponsor](https://knip.dev/sponsors)._

Add JUnit reporter

Is there still interest in wrapping up this pull request?

Add JUnit reporter

I feel like we could improve the output a bit. For instance, I see this in one section: > json.Unused devDependencies package ⋁ > > 1 Unused devDependencies - postcss-scss...

VS Code problemMatcher-friendly reporter

> The output format of knip isn't the best for parsing. Did you see the JSON reporter? https://knip.dev/features/reporters#json > This works pretty well: Nice! Great to see Knip used like...

Code formatting turns exactly 12 spaces into a line break

Could someone maybe try the same without reveal-md, but with only reveal.js?

Code formatting turns exactly 12 spaces into a line break

Thanks!

Do not assume node_modules path

It's not pretty, but I don't see how this could be different across package managers? This is navigating within a single package.

Do not assume node_modules path

Ah yes, PnP. PR is welcome :)