Skip to content

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

Merged
serhalp merged 4 commits into
npmx-dev:mainfrom
cylewaitforit:knip-5.86.0
Mar 8, 2026
Merged

chore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from config#1980
serhalp merged 4 commits into
npmx-dev:mainfrom
cylewaitforit:knip-5.86.0

Conversation

@cylewaitforit

Copy link
Copy Markdown
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

vercelBot commented Mar 7, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devReadyReadyPreview, CommentMar 8, 2026 3:51am
npmx.devReadyReadyPreview, CommentMar 8, 2026 3:51am
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
npmx-lunariaIgnoredIgnoredMar 8, 2026 3:51am

Request Review

@codecov

codecovBot commented Mar 7, 2026

Copy link
Copy Markdown

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
Copy Markdown
Contributor

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 nameStatusExplanation
Description check✅ PassedThe 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.

@cylewaitforitcylewaitforit changed the title chore(knip): remove unneeded entry and ignoreDependencies from knip configchore(deps): remove unneeded entry and ignoreDependencies from knip configMar 7, 2026
@cylewaitforitcylewaitforit changed the title chore(deps): remove unneeded entry and ignoreDependencies from knip configchore(deps): bump knip to 5.86.0 and remove unneeded entry and ignoreDependencies from configMar 7, 2026
@cylewaitforitcylewaitforit changed the title chore(deps): bump knip to 5.86.0 and remove unneeded entry and ignoreDependencies from configchore(deps): update devdependency knip to 5.86.0 and remove unneeded entry and ignoreDependencies from configMar 7, 2026
@serhalp
serhalp added this pull request to the merge queueMar 8, 2026
Merged via the queue into npmx-dev:main with commit e91c760Mar 8, 2026
21 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Mar 8, 2026
Sign up for freeto 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.

3 participants

@cylewaitforit@serhalp@alexdln