Gulp source adapter, release 1.0

[robwierzbowski]

1.0 will implement the plugin as a vinyl stream source, similar to a vinyl adapter, but only implementing the src part, and using gulp.watch and gulp.dest for the rest.

Interface:

gulp.task('styles', function () {
  return rubySass(
    // This replaces `update` or `file:file` as the source. 
    // The appropriate Sass command is fired 
    // based on whether a file or directory is passed.
    'app/styles', {
        sourcemap: true,
        style: 'expanded',
        precision: 10
    })
    .pipe($.autoprefixer(['last 2 versions']))
    .pipe(gulp.dest('.tmp/styles'))
    .pipe($.size({title: 'styles'}));
});

Because it's basically spawn that pipes into a gulp stream I think the only things we'd have to touch in the output would be sourcemaps (of course) and maybe error handling.

We would lose the ability to add the plugin at any point in the pipe. But aside from linting, which could be a separate task, I've seen no common reason to transform the Sass files before compiling them to CSS. The maintenance / allowing 90% of users to use Sass like they would on the cli or with Grunt outweighs supporting those edge cases.

• Implement as vinyl stream source with update
• Decide on error handling, re-implement logging
• Add gulp-sourcemap support for inline and none
• Implement as vinyl stream source with update directory compilation
• Add single file compilation
• Add gulp-sourcemap support for file
• Add gulp-sourcemap tests
• Add gulp-sourcemap recipes
• Allow parallel calls to the plugin without jumble of CSS files Instead of making this automatic I expanded documentation of the container option, added a recipe.
• Update readme with changes

Feature level releases after 1.0 (or if I have time during the alpha)

• Support file globbing
• Add smart src property that delegates between dirs, files, and file globs
• Document a working incremental build recipe

[sindresorhus]

Nice!

We would lose the ability to add the plugin at any point in the pipe.

Should be mentioned in the readme ;)

[robwierzbowski]

For sure. I'll tidy up the readme last, to get everything that changes.

On Sunday, September 28, 2014, Sindre Sorhus notifications@github.com
wrote:

Nice!

We would lose the ability to add the plugin at any point in the pipe.

Should be mentioned in the readme ;)

—
Reply to this email directly or view it on GitHub
#137 (comment)
.

Rob Wierzbowski
@robwierzbowski http://twitter.com/#!/robwierzbowski
http://github.com/robwierzbowski
http://robwierzbowski.com

[UltCombo]

We would lose the ability to add the plugin at any point in the pipe. But aside from linting, which could be a separate task, I've seen no common reason to transform the Sass files before compiling them to CSS.

I just found an issue with that, namely incremental builds with gulp-watch. E.g., this is a (IMO) common use case that would no longer work:

gulp.task('default', function() {
    var srcGlob = 'scss/*.scss';
    return gulp.src(srcGlob)
        .pipe(watch(srcGlob, function(files) {
            return files
                .pipe(sass())
                .pipe(gulp.dest('css'));
    }));
});

[robwierzbowski]

Good point. For simple tasks if we get sass-cache working we'd be ok. There are a bunch of gulp incremental filtering tasks — I wonder if one will work with this style plugin.

The adapter approach solves so many problems I think even with only a partial support case it's worth it.

[robwierzbowski]

Added @sindresorhus's readme note and @UltCombo's incremental build note to the issue tasks.

[ybian]

Waiting for this...

[robwierzbowski]

Sourcemaps working for inline. Next in priority is file, then allowing compilation of a single file, then documenting or implementing an incremental/cached build. After that I'll roll an aplha and start pinging people to test it while I work on other features.

[UltCombo]

Yay nice work @robwierzbowski! 😄

[robwierzbowski]

Re: sourcemap integration, here's a related Sass issue.

sass/sass#1485

The answer to this will change how we deal with sourcemaps.

[robwierzbowski]

OK, that's it. Ready to make 1.0 real and go from there.

[robwierzbowski]

Published 1.0.0. OK, now to take care of these: https://github.com/sindresorhus/gulp-ruby-sass#important

[sindresorhus]

Woot! Awesome work @robwierzbowski :)

[jmcbee]

ahaha