Worth refactoring to ES5? about avl HOT 4 CLOSED

that's what d3 is using
https://github.com/d3/d3-geo-projection/blob/master/package.json#L24

{
  ...
  "main": "build/d3-geo-projection.js",
  "module": "index",
  "jsnext:main": "index"
}

from avl.

well, I just used the class syntax sugar for readability, the rest I wanted to keep under full control. Maybe you are right, but I don't know how to make it export es6 module as well

from avl.

I just used the class syntax sugar for readability

Definitely sugar 😄 It's really not too bad writing it in pure ES5.

make it export es6 module as well

Also struggling with that part, your library would export as a CommonJS library, and for those using Rollup they would need to use rollup-plugin-commonjs in their config (which isn't the end of the world since a lot of modules still export as CommonJS). All the other bundlers don't have any issues with CommonJS libraries (babel, browserify, webpack).

from avl.

👍 Oh I like it! I might try to use that for some of my modules 😄

from avl.