Performance Issues - Githubissues

mariocasciaro / gulp-concat-css

Concatenates css files, bubbling up import statements (as per the standard), and optionally rebasing urls and inlining local import statements.

MIT License

78 stars 19 forks source link

Performance Issues #15

Open jlukic opened 9 years ago

jlukic commented 9 years ago

I'm seeing drastic performance difference when using gulp-concat-css over gulp-concat

15-30s to build 600kb file on quad-core versus with gulp-concat-css
1-2s with gulp-concat

I know rework is necessary, but perhaps there's some room for performance gains? https://github.com/Semantic-Org/Semantic-UI/issues/1464

mariocasciaro commented 9 years ago

May I know some more details about your sources? How many files are you concatenating? What's the typical dependency depth?

jlukic commented 9 years ago

Concatenating about 50 css files
Not inlining any @import rules
Moving a single remote @import to the top of the css.

  return gulp.src(output.uncompressed + '**/' + componentGlob + config.ignoredFiles + '.css')
    .pipe(plumber())
    .pipe(concatCSS('semantic.css'))
      .pipe(gulp.dest(output.packaged))
      .pipe(print(log.created))
  ;

mariocasciaro commented 9 years ago

Version 1.2.0 includes a performance fix (20% improvement). Please, let me know what's the improvement for you project.