Do we need sourcemaps in production build?

yeoman / generator-webapp

A gulp.js generator for modern webapps

3.77k stars 708 forks source link

Do we need sourcemaps in production build? #599

Closed Trost123 closed 5 years ago

Trost123 commented 8 years ago

I just noticed that main.css (and main.js) have sourcemaps embedded in dest folder. Do we need them on production? If not, it can be fixed by modifying 2 lines in scripts and styles tasks:

    .pipe($.if(dev, $.sourcemaps.init()))
    .pipe($.if(dev, $.sourcemaps.write()))

silvenon commented 8 years ago

Great question, I don't think we need them in production. Feel free to open a PR 😉

silvenon commented 5 years ago

Fixed in v4.0.0-0!