Removed stale secretlint entries from knip ignoreDependencies#27947
Conversation
knip flagged these as stale configuration hints — it now resolves secretlint and @secretlint/* through .secretlintrc.json discovery, so the explicit ignoreDependencies entries are no longer needed.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughThis pull request removes unused configuration from 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
knip flagged these entries as stale (
Remove from ignoreDependenciesconfiguration hints) — it now resolvessecretlintand@secretlint/*through.secretlintrc.jsondiscovery, so the explicitignoreDependenciesentries are no longer needed.Test plan
pnpm knipno longer shows the configuration hintssecretlintand@secretlint/*are not flagged as unlisted/unused