Skip to content

Conversation

tbitai
Copy link
Contributor

@tbitai tbitai commented Aug 27, 2016

Needed to bundle with Rollup.

@stackfull
Copy link
Owner

I'm not familiar with rollup, but I don't think it's appropriate to set main to anything in 'dist' because it doesn't exist until built. Maybe the bower component
is what you want?

@tbitai
Copy link
Contributor Author

tbitai commented Aug 27, 2016

I'm sorry, you're right that this is not the place where main should be set. It should be in the NPM package (I guess one could use Bower via the bower-resolve Rollup plugin in a similar manner; I prefer NPM). The reason why I put this here is that I was confused by the fact that this "raw" package already contains a package.json.

I merged this in my NPM package. Please merge this in the official NPM package, too (and update it, so that my unofficial package won't be needed anymore). Thanks!

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