Skip to content

Commit

Permalink
Renaming "cache" to "cached" for clarity
Browse files Browse the repository at this point in the history
It is misleading to call this `cache` as it is loading the `grunt-cached` plugin.  All of the other variable names accurately represent the module that's being loaded.  Additionally, there is a `grunt-cache` plugin.  If I were to assume that people get things right most of the time, I'd try to first write my code to load `grunt-cache` because that variable is right twice and I would guess that the typo was just in the module name.
  • Loading branch information
fidian committed May 6, 2014
1 parent f68686d commit 335e210
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions docs/recipes/incremental-builds-with-concatenate.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@ var gulp = require('gulp'),
footer = require('gulp-footer'),
concat = require('gulp-concat'),
jshint = require('gulp-jshint'),
cache = require('gulp-cached'),
cached = require('gulp-cached'),
remember = require('gulp-remember');

var scriptsGlob = 'src/**/*.js';

gulp.task('scripts', function () {
return gulp.src(scriptsGlob)
.pipe(cache('scripts')) // only pass through changed files
.pipe(cached('scripts')) // only pass through changed files
.pipe(jshint()) // do special things to the changed files...
.pipe(header('(function () {')) // e.g. jshinting ^^^
.pipe(footer('})();')) // and some kind of module wrapping
Expand All @@ -35,4 +35,4 @@ gulp.task('watch', function () {
}
});
});
```
```

0 comments on commit 335e210

Please sign in to comment.