[Building-rollup] use per output plugins

[LarsDenBakker]

We should use rollup/rollup#3218 when its merged

[abdonrd]

Released! https://github.com/rollup/rollup/releases/tag/v1.27.0

[bennypowers]

What's the plan for this?

[LarsDenBakker]

Someone needs to dig in and analyze whether this feature can be useful for us.

My hope is that we can use the regular plugins entry for shared plugins, and then a per-output plugins for plugins that should run for the different output. That should make it a ton easier to extend the compatibility configuration for our users.

This is probably a breaking change, so we should also check whether there are any other pending breaking changes. I might have some soonish, I need to check And maybe @thepassle has some related to the new service worker stuff.

[thepassle]

I dont have any breaking changes planned wrt workbox. It would just be a shared plugin (for both outputs), so I think it can stay where it is

[bennypowers]

We discussed this on slack. One use case for use would be something like

export default {
  output: [
    { dir: 'dist/modern': plugins: [babel({ presets: ['@babel/preset-modules'] })] },
    { dir: 'dist/legacy': plugins: [babel({ presets: ['@babel/preset-env'] })] },
  ]
  // ...
}

[LarsDenBakker]

We will need this to be merged rollup/rollup-plugin-babel#333

[abdonrd]

We will need this to be merged rollup/rollup-plugin-babel#333

Merged! 😁

[LarsDenBakker]

They didn't release it yet though :/

[LarsDenBakker]

This was implemented in v1 of building-rollup.