thisandagain/sentiment

Bundle for browser usage?

Closed this issue · 4 comments

Nice work on this library! I found it helpful for one of my pet-projects, but when I tried to use it in the browser I saw you don't have a bundled version.

Have you considered using something like rollup to bundle you packages into a single deliverable?

I made a bundled version for my own use. You can check out my branch.

I made a few big changes, so I didn't want to submit a PR. I'm happy to discuss this further.

Hi @trainorpj. I think most folks generally use webpack or browserify to bundle sentiment into a front-end project. I'd be happy to accept a PR that adds some dev dependency(s) to enable having a standalone bundle for this purpose though.

Sure thing! I'll clean up my branch when I get a chance.

On reflection I'd prefer that folks use something like webpack or browserify if they would like to include this in a client-side application. This helps keep the complexity of this repo down and is fairly normative with the rest of the NPM ecosystem.