Skip to content

feat: keep inlineOnly clean with warning on unused#725

Merged
sxzz merged 2 commits intorolldown:mainfrom
jycouet:feat/keep-inlineOnly-lean
Jan 25, 2026
Merged

feat: keep inlineOnly clean with warning on unused#725
sxzz merged 2 commits intorolldown:mainfrom
jycouet:feat/keep-inlineOnly-lean

Conversation

@jycouet
Copy link
Contributor

@jycouet jycouet commented Jan 24, 2026

  • This PR contains AI-generated code, but I have carefully reviewed it myself. Otherwise, my PR may be closed.

Description

I started to use inlineOnly to be aware of things that goes into the bundle or not.
Today it's warning about things missing, but it will be nice to have also things to remove to keep this array clean.

Linked Issues

I read the Contributing Guide, but didn't create an issue before the PR, as I was playing with the code already.
Feel free to close the PR if it's not appropriate.

Additional context

PR tsdown bump in sv: sveltejs/cli#891

@netlify
Copy link

netlify bot commented Jan 24, 2026

Deploy Preview for tsdown ready!

Name Link
🔨 Latest commit ea80849
🔍 Latest deploy log https://app.netlify.com/projects/tsdown/deploys/6976383bc7c8fa00080e9742
😎 Deploy Preview https://deploy-preview-725--tsdown.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 24, 2026

Open in StackBlitz

npm i https://pkg.pr.new/tsdown@725
npm i https://pkg.pr.new/create-tsdown@725
npm i https://pkg.pr.new/tsdown-migrate@725

commit: ea80849

@sxzz sxzz merged commit 13f1c5d into rolldown:main Jan 25, 2026
16 checks passed
@jycouet
Copy link
Contributor Author

jycouet commented Jan 25, 2026

Nice, thank you 🎉

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