Skip to content

chore(deps): remove unused dependencies#773

Merged
danielroe merged 1 commit into
nuxt:mainfrom
bluwy:remove-unused
Feb 5, 2026
Merged

chore(deps): remove unused dependencies#773
danielroe merged 1 commit into
nuxt:mainfrom
bluwy:remove-unused

Conversation

@bluwy

@bluwy bluwy commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

🔗 Linked issue

n/a

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Since #645 migrated to fontless, these dependencies are moved into fontless instead, so we can remove them from this library.

Otherwise the same dependencies from both packages may be duplicated because of different versions: https://npmgraph.js.org/?q=%40nuxt%2Ffonts

@bluwy bluwy requested a review from danielroe as a code owner January 30, 2026 02:49
@pkg-pr-new

pkg-pr-new Bot commented Jan 30, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/fonts@773

commit: 5c22990

@coderabbitai

coderabbitai Bot commented Jan 30, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This pull request removes six runtime dependencies (css-tree, esbuild, fontaine, jiti, magic-string, ohash) and one development dependency (@types/css-tree) from package.json. No new dependencies are introduced, and no scripts or configuration logic are modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and accurately describes the main change: removing unused dependencies from package.json after migration to fontless.
Description check ✅ Passed The description is directly related to the changeset, explaining the context of the dependency removal and referencing the migration to fontless.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.78%. Comparing base (d551db7) to head (5c22990).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #773   +/-   ##
=======================================
  Coverage   66.78%   66.78%           
=======================================
  Files           9        9           
  Lines         286      286           
  Branches       64       64           
=======================================
  Hits          191      191           
  Misses         78       78           
  Partials       17       17           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ivanm696

ivanm696 commented Feb 3, 2026

Copy link
Copy Markdown

Screenshot_20260203_030312_com android chrome

@danielroe danielroe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you ❤️

@danielroe danielroe merged commit c60e4e3 into nuxt:main Feb 5, 2026
7 checks passed
@bluwy bluwy deleted the remove-unused branch February 5, 2026 14:14
@bluwy bluwy mentioned this pull request Feb 5, 2026
6 tasks
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.

4 participants