Skip to content

Commit 152d772

Browse files
author
John F. Mercer
committed
Fixes gulp security task
* Replaces the now-deprecated requireSafe with gulp-nsp * Restores security task to gulp default task * Minor formatting cleanup
1 parent 7f43062 commit 152d772

File tree

2 files changed

+13
-11
lines changed

2 files changed

+13
-11
lines changed

Gulpfile.js

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
'use strict';
22

3-
var gulp = require('gulp');
4-
var Server = require('karma').Server;
5-
var coveralls = require('gulp-coveralls');
6-
var jshint = require('gulp-jshint');
7-
var jscs = require('gulp-jscs');
3+
var gulp = require('gulp');
4+
var Server = require('karma').Server;
5+
var coveralls = require('gulp-coveralls');
6+
var jshint = require('gulp-jshint');
7+
var jscs = require('gulp-jscs');
8+
var nsp = require('gulp-nsp');
89
var runSequence = require('run-sequence');
9-
var gulpRequireSafe = require('gulp-requiresafe');
1010

1111
/*
1212
* PLEASE NOTE: run-sequence is a
@@ -46,9 +46,12 @@ gulp.task('style', function() {
4646
});
4747

4848
gulp.task('security', function(cb) {
49-
gulpRequireSafe({package: __dirname + '/package.json', stopOnError: true}, cb);
49+
nsp({
50+
package: __dirname + '/package.json',
51+
stopOnError: true
52+
}, cb);
5053
});
5154

5255
gulp.task('default', function(cb) {
53-
runSequence('test', ['lint', 'style', 'coveralls'], cb);
56+
runSequence('test', ['lint', 'style', 'coveralls', 'security'], cb);
5457
});

package.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
"gulp-coveralls": "^0.1.4",
3535
"gulp-jscs": "^3.0.2",
3636
"gulp-jshint": "^1.12.0",
37+
"gulp-nsp": "^2.0.1",
3738
"jasmine-core": "^2.3.4",
3839
"karma": "^0.13.14",
3940
"karma-coverage": "^0.5.3",
@@ -43,9 +44,7 @@
4344
"phantomjs": "^1.9.18",
4445
"run-sequence": "^1.1.4"
4546
},
46-
"dependencies": {
47-
"gulp-requiresafe": "^1.1.2"
48-
},
47+
"dependencies": {},
4948
"keywords": [
5049
"angular",
5150
"angularjs",

0 commit comments

Comments
 (0)