search

bublejs / buble

https://buble.surge.sh
MIT License
871 stars 64 forks source link

Removed acorn as a rollup external to avoid issues with conflicting v… #160

Closed alansouzati closed 6 years ago

alansouzati commented 6 years ago

…ersions.

Fixes https://github.com/Rich-Harris/buble/issues/159

adrianheine commented 6 years ago

This should also fix #158.

onigoetz commented 6 years ago

Does this change mean that acorn will be bundled with buble directly ?

adrianheine commented 6 years ago

That's what it means, and I don't like it and I don't understand why npm does not do this one thing it's supposed to do. @alansouzati Which npm version do you use?

onigoetz commented 6 years ago

The question isn't adressed to me, but I use Yarn (1.7.0) and have the same issue

alansouzati commented 6 years ago

I’m using the latest NPM and yarn, and both yield the same issue

adrianheine commented 6 years ago

I'm closing this in favor of #161 and #162.