Skip to content

Commit a0cd82c

Browse files
committed
clean up
1 parent 732e8ae commit a0cd82c

File tree

7 files changed

+24
-54
lines changed

7 files changed

+24
-54
lines changed

examples/copy.js

+9-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,15 @@
11
'use strict';
22

3+
var path = require('path');
34
var copy = require('..');
4-
var dest = 'test/actual/blah/foo';
5-
var opts = {cwd: 'test/fixtures'};
5+
var log = require('log-ok');
6+
var cwd = path.resolve.bind(path, __dirname, '..');
7+
var dest = cwd('test/actual/blah/foo');
8+
var opts = {cwd: cwd('test/fixtures')};
69

7-
copy('*.txt', dest, opts, function(err) {
10+
copy('*.txt', dest, opts, function(err, files) {
811
if (err) return console.error(err);
9-
console.log('done!');
12+
files.forEach(function(file) {
13+
log(file.relative);
14+
})
1015
});

gulpfile.js

+7
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,19 @@
22

33
var del = require('delete');
44
var gulp = require('gulp');
5+
var unused = require('gulp-unused');
56
var copy = require('./');
67

78
gulp.task('default', function(cb) {
89
copy('test/fixtures/**/*.*', 'test/actual', cb);
910
});
1011

12+
gulp.task('unused', function() {
13+
var keys = Object.keys(require('./lib/utils.js'));
14+
return gulp.src(['index.js', 'lib/*.js'])
15+
.pipe(unused({keys: keys}))
16+
});
17+
1118
gulp.task('del', function(cb) {
1219
del('test/actual', cb);
1320
});

lib/contents.js

-41
This file was deleted.

lib/once.js

+4-3
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,10 @@ module.exports = function(fn) {
1111
}
1212
var result;
1313
return function() {
14-
if (fn.called) return result;
15-
result = fn.apply(this, arguments);
16-
fn.called = true;
14+
if (!fn.called) {
15+
fn.called = true;
16+
result = fn.apply(this, arguments);
17+
}
1718
return result;
1819
};
1920
};

lib/recurse.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
'use strict';
22

33
var path = require('path');
4-
var contents = require('./contents');
54
var utils = require('./utils');
65

76
/**
@@ -26,7 +25,7 @@ function recurse(cwd, callback) {
2625
// temporary. errors are currently silenced in file-stat,
2726
// once that's updated we'll remove this
2827
try {
29-
fs.readdirSync(res.path);
28+
utils.fs.readdirSync(res.path);
3029
} catch (err) {
3130
cb(err);
3231
return;

lib/utils.js

-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ require('matched', 'glob');
2424
require('mkdirp');
2525
require('resolve-dir', 'resolve');
2626
require('to-file');
27-
require('vinyl', 'File');
2827
require = fn;
2928

3029
utils.stat = function(file) {

package.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -37,15 +37,15 @@
3737
"matched": "^0.4.1",
3838
"mkdirp": "^0.5.1",
3939
"resolve-dir": "^0.1.0",
40-
"to-file": "^0.2.0",
41-
"vinyl": "^1.1.1"
40+
"to-file": "^0.2.0"
4241
},
4342
"devDependencies": {
4443
"assert-fs": "^0.1.0",
4544
"assert-path": "^0.1.0",
4645
"delete": "^0.3.2",
4746
"gulp": "^3.9.1",
4847
"gulp-format-md": "^0.1.9",
48+
"gulp-unused": "^0.1.2",
4949
"is-buffer": "^1.1.3",
5050
"look-up": "^0.8.3",
5151
"minimist": "^1.2.0",
@@ -68,7 +68,7 @@
6868
],
6969
"verb": {
7070
"run": true,
71-
"toc": false,
71+
"toc": true,
7272
"layout": "default",
7373
"tasks": [
7474
"readme"

0 commit comments

Comments
 (0)