esm compatibility for three-shaking with Rollup

[frederikhors]

I would like to use Rollup to three-shaking this amazing code (e.g. I'm using only fadeIn).

Can you export default your code making it esm compatible?

Actually I'm using it like this:

import Turbolinks from 'turbolinks'
import 'turbolinks-animate'

Turbolinks.start()

document.addEventListener('turbolinks:load', () => window.TurbolinksAnimate.init())

Am I wrong in my desire?

Thanks.

[jonhue]

Hmm... is this really necessary? The way you're using it right now works and there is no code that could be eliminated by three-shaking.
Don't get me wrong. Ideally, we would use ES6 syntax. But the problem is backward compatibility.
Moving to ES6 syntax would mean we'd have to use Babel. Right now I don't have the time to do the migration considering that the code in its current state still works. If you have time feel free to have a go at it. I'm always happy to review pull requests 🙂.