Skip to content

refactor: remove unused dependencies - #286

Merged
43081j merged 2 commits into
e18e:mainfrom
AFCMS:remove_unused
Nov 5, 2026
Merged

43081j merged 2 commits into
e18e:mainfrom
AFCMS:remove_unused

Conversation

@AFCMS

@AFCMS AFCMS commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Remove unused dependencies reported by knip.

@github-actions

github-actions Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

📊 Dependency Size Changes

Warning

This PR adds 1.3 MB of new dependencies, which exceeds the threshold of 100 kB.

📦 Package 📏 Size
@rolldown/binding-linux-x64-gnu@1.2.14 → @rolldown/binding-linux-x64-gnu@1.2.16 1.3 MB
@types/semver@7.8.0 -24.7 kB
rolldown@1.2.14 → rolldown@1.2.16 4.5 kB
vite@8.3.4 → vite@8.3.6 901 B
postcss@8.5.30 → postcss@8.5.32 316 B
@oxc-project/types@0.154.0 → @oxc-project/types@0.156.0 0 B

Total size change: 1.3 MB

@pkg-pr-new

pkg-pr-new Bot commented Nov 5, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@e18e/cli@286

commit: 984cb4c

@43081j
43081j merged commit 13680a1 into e18e:main Nov 5, 2026
6 checks passed
@AFCMS
AFCMS deleted the remove_unused branch November 5, 2026 10:20
@AFCMS

AFCMS commented Nov 5, 2026

Copy link
Copy Markdown
Contributor Author

No idea why the build failed like this, could it be a npm CLI version mismatch?

@43081j

43081j commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

it was just an already broken lockfile afaict. seems fine now 👍

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