search

crysalead-js / dom-layer

Virtual DOM implementation.
MIT License
30 stars 1 forks source link

Replace browserify with rollupjs #68

Closed kolya-ay closed 8 years ago

kolya-ay commented 8 years ago

It seems that it's better choice for libraries at least. It generates almost no boilerplate

jails commented 8 years ago

Looks interesting! Anyhow the provided dist file is only provided to make it easier to use this library in some online tool like jsFiddle. People can still use rollupjs or the tool you want to bunde up all of your dependencies through node.

kolya-ay commented 8 years ago

I think it still makes sense as rollupjs not only provides the solution but also proposes a standard for having best of two worlds

jails commented 8 years ago

dom-layer doesn't define the "main" entry as "dist/dom-layer.js" in its package.json. index.js is still the entry point so the generated build will depends on your tooling. So if you are using rollupjs in your project, dom-layer will be build using rollupjs. dist/dom-layer.js won't be included as is. dist/dom-layer.js is only present to make some JSFiddle without to much headach.