Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add rollup-plugin-multi-input #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

logustra
Copy link

@logustra logustra commented Nov 20, 2020

Awesome Contribution Checklist:

  • I have read, and re-read the Contributing Guidelines
  • I have searched to ensure the suggested item doesn't exist on this list
  • This PR contains only one item

Please Provide a Link A Repository for Your Addition

https://github.com/alfredosalzillo/rollup-plugin-multi-input

Please Describe Your Addition

A rollup plugin to bundle modular libraries with sub directories.

@shellscape
Copy link
Contributor

Can you explain how this plugin differs from the official https://github.com/rollup/plugins/tree/master/packages/multi-entry plugin?

@logustra
Copy link
Author

@shellscape
multi-entry cannot generate output dynamically
let's say I've 5 input but I always got 1 output

@alfredosalzillo
Copy link

https://github.com/alfredosalzillo/rollup-plugin-multi-input can be used to bundle modular libraries.

The https://github.com/rollup/plugins/tree/master/packages/multi-entry can't because it only allow using multiple entries but doesn't assure the output directories tree is preserved.

@shellscape
Copy link
Contributor

Thanks for that info. This is going to be inherently confusing to users based on the name, due to the existing official plugin. Some questions:

  • Is there a more apt name for this module that provides distinction from the official plugin of similar name?
  • Is there an opportunity contribute to the multi-entry plugin to enable the options and behaviors that rollup-plugin-mutli-input provides?

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.

3 participants