It's failing due to webpack version.

As a side note, later, we should replace webpack by rollup

I wonder if just rolling back babel-loader to 6.2.0 would be enough to iron it out. That one isn't involved in the build so I probably should have left it alone.

@marionettejs/marionette-core if 2nd thumb'd squash and merge

We need to make sure this has named exports

I think it can be merged as is.

The addition of named exports can be added later since means a bigger change and needs some discussion.

Radio public API is basically Channel class, channel function , reset, tuneIn, tuneOut, log, debugLog

While in Marionette migrating everything to named exports and killing the default exports was straight, in Radio doing the same will be a big breaking change.

By far the most used API in channel function, used like:

import Radio from 'backbone.radio'

Radio.channel('yy').request('xx')

By migrating to named exports like it would need to be changed to

import {channel} from 'backbone.radio'

channel('yy').request('xx')

There's some issues:

• channel is not a good name for a function, looks more like a variable
• channel and Channel would be exported, adding confusion
• debugLog could not be override

So migrating to named exports is possible but will need some API design changes that at this time (just before releasing marionette v4) i don't think is doable.

I propose to merge this and open a new issue for named export