Skip to content

chore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config#1980

Open
cylewaitforit wants to merge 4 commits intonpmx-dev:mainfrom
cylewaitforit:knip-5.86.0
Open

chore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config#1980
cylewaitforit wants to merge 4 commits intonpmx-dev:mainfrom
cylewaitforit:knip-5.86.0

Conversation

@cylewaitforit
Copy link
Contributor

🔗 Linked issue

🧭 Context

When running knip it has configuration hints for what can be removed from the config file.

Configuration hints (6)
@intlify/shared               knip.ts  Remove from ignoreDependencies
@nuxt/ui                docs  knip.ts  Remove from ignoreDependencies
tailwindcss             docs  knip.ts  Remove from ignoreDependencies
app/router.options.ts!        knip.ts  Remove redundant entry pattern
app/app.vue!                  knip.ts  Remove redundant entry pattern
app/error.vue!                knip.ts  Remove redundant entry pattern

📚 Description

This removes those hints by updating to knip 5.86.0 and deletes the config lines that were throwing the hints.

@vercel
Copy link

vercel bot commented Mar 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Mar 8, 2026 3:51am
npmx.dev Ready Ready Preview, Comment Mar 8, 2026 3:51am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Mar 8, 2026 3:51am

Request Review

@codecov
Copy link

codecov bot commented Mar 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 650a7a17-efa5-4d55-9ed3-71b67f23739d

📥 Commits

Reviewing files that changed from the base of the PR and between 0d952fa and 53e06c0.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • knip.ts
  • package.json

📝 Walkthrough

Walkthrough

This 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

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly explains the context, problem, and solution by detailing the knip configuration hints and how they are addressed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cylewaitforit cylewaitforit changed the title chore(knip): remove unneeded entry and ignoreDependencies from knip config chore(deps): remove unneeded entry and ignoreDependencies from knip config Mar 7, 2026
@cylewaitforit cylewaitforit changed the title chore(deps): remove unneeded entry and ignoreDependencies from knip config chore(deps): bump knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config Mar 7, 2026
@cylewaitforit cylewaitforit changed the title chore(deps): bump knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config chore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants