gulp icon indicating copy to clipboard operation
gulp copied to clipboard

Docs: Refactor Rollup recipe

Open silvenon opened this issue 3 years ago • 8 comments

Nothing extra is needed to use Rollup with gulp, so this refactors the recipe to use Rollup's JavaScript API directly.

silvenon avatar Oct 18 '20 10:10 silvenon

Alternatively, I could add this next to the rollup-stream example, if you think it's useful to keep that one. But it's best to use tools directly unless it's inconvenient.

silvenon avatar Oct 18 '20 10:10 silvenon

I believe this recipe should describe both:

  1. Rollup's normal JS API, which should be preferred if you don't need to apply other stream transforms
  2. The new official stream API: https://www.npmjs.com/package/@rollup/stream if you do need to apply more stream transforms.

I think we can include both in this recipe.

phated avatar Oct 19 '20 18:10 phated

@silvenon I also think this recipe should just be called "Bundling JavaScript", since it will be the recommended way to include bundling inside your gulpfiles.

phated avatar Oct 21 '20 19:10 phated

I started working on this again. My plan is to rename this to "Using Module Bundlers" instead because I also want to add webpack and esbuild, both of which support other bundling other assets, not just JavaScript. I'll finish Rollup in the following days and as separate PRs I'll add other bundlers as well.

silvenon avatar Nov 05 '20 08:11 silvenon

@silvenon I think that is a great plan! Thank you for updating us.

phated avatar Nov 05 '20 18:11 phated

@silvenon are you still planning to complete the above changes?

phated avatar Apr 02 '21 23:04 phated

No, sorry, I lost my drive for some reason. 😔 I'll open a new PR once I get it back.

silvenon avatar Apr 24 '21 10:04 silvenon

Thanks for letting me know! I'd like to re-open and pick this up myself. And thank you for the work you put in here already ❤️

phated avatar May 03 '21 19:05 phated