Skip to content

fix: perserve directives#511

Open
kricsleo wants to merge 2 commits intounjs:mainfrom
kricsleo:fix/preserve-directives
Open

fix: perserve directives#511
kricsleo wants to merge 2 commits intounjs:mainfrom
kricsleo:fix/preserve-directives

Conversation

@kricsleo
Copy link
Copy Markdown
Member

resolves #292

@pi0
Copy link
Copy Markdown
Member

pi0 commented Feb 26, 2025

Thanks for PR, it is nice feature, however I'm wondering if it is worth to add a new rollup dependency for this (while the main usage of directives, is mainly for modules like react components which have to be used via mkdist/ builder mode not rollup)

Do you know any specific case that directives in "single bundle" are beneficial and mkdist cannot be used?

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.

Preserving directives

2 participants