Skip to content

feat: add banner, footer options#338

Merged
sxzz merged 7 commits intorolldown:mainfrom
gwansikk:feature/banner-footer
Jul 18, 2025
Merged

feat: add banner, footer options#338
sxzz merged 7 commits intorolldown:mainfrom
gwansikk:feature/banner-footer

Conversation

@gwansikk
Copy link
Contributor

@gwansikk gwansikk commented Jun 21, 2025

Description

Note

Hello. I'm an enthusiast of tsup and have great interest in the tsdown project.
This PR is merely a suggestion and I submitted it with the intention of having a discussion with you.
If this PR doesn't align with the direction of tsdown, please feel free to close it.

tsdown is a project inspired by tsup, and in fact, many of its configuration structures and core philosophies are designed similarly.
For that reason, tsup users who switch to tsdown inevitably expect a similar developer experience (DX).
I believe the banner and footer features fall within that context. These are widely used in various bundlers such as Rollup, esbuild, and webpack, and have been supported in tsup as well.

However, since tsdown currently uses rolldown internally, although the functionality itself is easy to implement, the configuration method has changed.
As a result, existing tsup users face an unnecessary entry barrier by having to look up documentation again or modify their existing configurations.
I believe this isn't just an interface difference, but a factor that causes friction during migration.

To become a project that tsup users can naturally transition to, I think it's important to maintain as much functional parity and consistency in configuration interface as possible. (Just my opinion.)
This PR, as part of that, proposes adopting the same banner/footer configuration style as tsup.

Of course, due to rolldown's structure, the future direction may change.
If so, I think we could proceed by separating it through a breaking change via versioning later on.
However, at this stage, encouraging user adoption with low friction and then shaping the direction seems like a more strategic approach.
The fact that users can transition with a similar API and minimal configuration changes could offer great convenience.

Linked Issues

closed: #288

@netlify
Copy link

netlify bot commented Jun 21, 2025

Deploy Preview for tsdown ready!

Name Link
🔨 Latest commit c213972
🔍 Latest deploy log https://app.netlify.com/projects/tsdown/deploys/687a19c79aafbd00087af8f2
😎 Deploy Preview https://deploy-preview-338--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 Jun 30, 2025

Open in StackBlitz

npm i https://pkg.pr.new/tsdown@338

commit: c213972

@gwansikk gwansikk changed the title feat: add banner, footer option feat: add banner, footer options Jul 2, 2025
@gwansikk gwansikk marked this pull request as ready for review July 2, 2025 16:36
@gwansikk
Copy link
Contributor Author

@sxzz I want to hear your opinion and continue the task based on it (Please share your thoughts!)

@sxzz sxzz force-pushed the feature/banner-footer branch from 81add3a to 434d6a4 Compare July 18, 2025 09:40
@sxzz sxzz enabled auto-merge (squash) July 18, 2025 09:54
@sxzz sxzz merged commit 54e21e1 into rolldown:main Jul 18, 2025
13 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.

Add banner and footer options for output file customization

2 participants