Skip to content

chore: remove unused dependencies - #3345

Merged
ryansolid merged 2 commits into
solidjs:nextfrom
nickshiro:chore/remove-unused-dependencies
Sep 10, 2026
Merged

chore: remove unused dependencies#3345
ryansolid merged 2 commits into
solidjs:nextfrom
nickshiro:chore/remove-unused-dependencies

Conversation

@nickshiro

Copy link
Copy Markdown

Summary

Remove dependencies that are no longer used by the workspace or Babel plugin.

From the root devDependencies:

  • @babel/cli — no package script invokes the Babel CLI anymore.
  • @rollup/plugin-json — no Rollup configuration imports the plugin.
  • coveralls — coverage is uploaded through coverallsapp/github-action.
  • rollup-plugin-copy — the last Rollup usage was removed in 2022.

From @solidjs/babel-plugin:

  • validate-html-nesting — the compiler validation implementation uses parse5 directly and does not import this package.

The lockfile was regenerated, removing 62 packages in total, including the obsolete Coveralls dependency tree.

How did you test this change?

  • pnpm install --frozen-lockfile --ignore-scripts.
  • pnpm --filter @solidjs/babel-plugin test:
    - 28 test files passed.
    - 255 tests passed.
  • pnpm build:
    - 25 tasks completed successfully.
    -Ran Prettier against both modified package manifests.
  • git diff --check.

@changeset-bot

changeset-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 993f995

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codspeed-hq

codspeed-hq Bot commented Sep 10, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 151 untouched benchmarks


Comparing nickshiro:chore/remove-unused-dependencies (993f995) with next (7f416cf)

Open in CodSpeed

@ryansolid
ryansolid merged commit fd58a92 into solidjs:next Sep 10, 2026
6 checks passed
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