nDmitry / grunt-postcss

Apply several post-processors to your CSS using PostCSS.
MIT License
419 stars 57 forks source link

Add sequential options which allows process files one after another #105

Closed VitaliyR closed 6 years ago

VitaliyR commented 7 years ago

Hi!

Using grunt-postcss on big project with a lot of CSS files (about 130 CSS files with 40MB total size) could cause memory out problem because of running processors in async mode.

This PR contains possibility to switch that mode to sync - so CSS files would be processed one after another, sequentially. It also saves a lot of RAM - GC will correctly find and kill that already processed files stuffs (source code, result object etc).

By default, this option set to false, so nothing will be changed for projects already using it.

Maybe you have some notices how we can refactor this changes, if needed?

VitaliyR commented 7 years ago

Seems this repo is abandoned?

C-Lodder commented 6 years ago

@VitaliyR You're welcome to submit this PR against the fork I've made: https://github.com/C-Lodder/grunt-postcss and I'll happily merge it.

If you choose to do so, please don't forget to add the tests too. Thank you

VitaliyR commented 6 years ago

Moving to @C-Lodder fork.