chore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config#1980
chore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config#1980cylewaitforit wants to merge 4 commits intonpmx-dev:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR updates the knip dependency management tool configuration and its version. In knip.ts, three files are removed from the root workspace entry (app/router.options.ts, app/app.vue, and app/error.vue), a runtime dependency comment for '@intlify/shared' is removed from ignoreDependencies in the root workspace, and the docs workspace ignoreDependencies list is simplified by removing several entries including '@nuxt/ui' and 'tailwindcss'. The knip package version in package.json is bumped from 5.85.0 to 5.86.0. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
🔗 Linked issue
🧭 Context
When running knip it has configuration hints for what can be removed from the config file.
📚 Description
This removes those hints by updating to knip 5.86.0 and deletes the config lines that were throwing the hints.