Fix yarn run dist-es to export idiomatic ES modules
ClosedPublic

Authored by appsforartists on Aug 8 2017, 5:23 PM.

Diff Detail

Repository
R51 wobble
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
appsforartists created this revision.Aug 8 2017, 5:23 PM
Restricted Application added a reviewer: skevy. Β· View Herald TranscriptAug 8 2017, 5:23 PM
Restricted Application completed building Restricted Buildable.Aug 8 2017, 5:25 PM
skevy accepted this revision.Aug 8 2017, 5:45 PM

This is fine.

I was kind of just giving this a try. Rollup is supposed to emit a bundle that uses ES modules inside of it, and is able to be consumed by webpack, etc. just as if the tool were consuming a folder of modules. Perhaps it doesn't work well though -- was just an experiment.

This revision is now accepted and ready to land.Aug 8 2017, 5:45 PM

No worries.

My concern is that it emits a bunch of polyfill, which you don't need if you're in an environment that speaks modules. I suspect that Closure will have a better time minifying an actual class than a polyfilled/obfuscated one.

This revision was automatically updated to reflect the committed changes.