foliojs / fontkit

An advanced font engine for Node and the browser
1.46k stars 219 forks source link

Drop babel-runtime requirement #204

Closed mclark-newvistas closed 2 years ago

mclark-newvistas commented 5 years ago

Would it be possible to drop babel-runtime from the package.json when targeting nodejs? I am running node 8.4.0 in production right now, and I don't believe babel offers anything which node lacks. Perhaps make it an optional dependency?

I ask because babel-runtime is indirectly causing issues by including core-js, which itself is spamming my terminal. Ideally we can break the dependency here, which should remove spam from my terminal. See babel/babel#10253 for an example of the terminal output, and why they are not able to break the dependency at that level.

blikblum commented 5 years ago

196 should make it