Skip to content

Commit fd7082f

Browse files
Added codeclimate in project, configs, fix eslint on js files and add npm
1 parent 2acd33c commit fd7082f

File tree

14 files changed

+896
-92
lines changed

14 files changed

+896
-92
lines changed

.codeclimate.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
---
2+
engines:
3+
duplication:
4+
enabled: true
5+
config:
6+
languages:
7+
- javascript
8+
eslint:
9+
enabled: true
10+
fixme:
11+
enabled: true
12+
ratings:
13+
paths:
14+
- "**.js"
15+
exclude_paths:
16+
- test/

.eslintignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
**/*{.,-}min.js

.eslintrc

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,12 @@
11
{
2+
"extends": "standard",
3+
"plugins": [
4+
"standard",
5+
"promise"
6+
],
7+
"parserOptions": {
8+
"ecmaVersion": 5
9+
},
210
"ecmaFeatures": {
311
"modules": true
412
},

index.js

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

3-
module.exports = require('./lib');
3+
module.exports = require('./lib')

lib/config.js

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

3-
const notify = require('gulp-notify');
4-
const bourbon = require('node-bourbon');
3+
const notify = require('gulp-notify')
4+
const bourbon = require('node-bourbon')
55

66
module.exports = {
77
sassConfig: {
@@ -20,7 +20,7 @@ module.exports = {
2020
},
2121
jsbeautifier: {
2222
indent_size: 2,
23-
indent_char: ' ',
23+
indent_char: ' '
2424
},
2525
autoprefixer: {
2626
browsers: [
@@ -38,8 +38,8 @@ module.exports = {
3838
},
3939
plumberErrorHandler: {
4040
errorHandler: notify.onError({
41-
title : 'Gulp',
42-
message : 'Error: <%= error.message %>'
41+
title: 'Gulp',
42+
message: 'Error: <%= error.message %>'
4343
})
4444
}
4545
}

lib/copy.js

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

3-
const merge = require('lodash').merge;
4-
const $ = require('gulp-load-plugins')();
3+
const merge = require('lodash').merge
4+
const $ = require('gulp-load-plugins')()
55

66
const defaults = {
77
taskname: 'copy',
88
src: './static/*',
99
dest: './dist/',
1010
size: false
11-
};
11+
}
1212

1313
module.exports = (gulp, options) => {
14-
const opts = merge({}, defaults, options);
15-
const taskname = opts.taskname;
16-
const size = opts.size;
14+
const opts = merge({}, defaults, options)
15+
const taskname = opts.taskname
16+
const size = opts.size
1717

1818
gulp.task(taskname, () => gulp.src(opts.src)
1919
.pipe(size ? $.size({ title: `Gulp task ${taskname}`, gzip: false, showFiles: true }) : $.util.noop())
20-
.pipe(gulp.dest(opts.dest)));
21-
};
20+
.pipe(gulp.dest(opts.dest)))
21+
}

lib/index.js

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

3-
module.exports = require('require-dir')();
3+
module.exports = require('require-dir')()

lib/sass.js

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

3-
const merge = require('lodash').merge;
4-
const config = require('./config');
5-
const $ = require('gulp-load-plugins')();
3+
const merge = require('lodash').merge
4+
const config = require('./config')
5+
const $ = require('gulp-load-plugins')()
66

77
const defaults = {
88
taskname: 'sass',
@@ -15,20 +15,20 @@ const defaults = {
1515
minify: false,
1616
beautifier: false,
1717
production: false
18-
};
18+
}
1919

2020
module.exports = (gulp, options) => {
21-
const opts = merge({}, defaults, options);
22-
const src = opts.src;
23-
const taskname = opts.taskname;
24-
const prefixer = opts.autoprefixer;
25-
const combinemq = opts.combinemq;
26-
const minify = opts.minify;
27-
const beautifier = opts.beautifier;
28-
const plumb = opts.plumb;
29-
const dest = opts.dest;
30-
const size = opts.size;
31-
const isProd = opts.production;
21+
const opts = merge({}, defaults, options)
22+
const src = opts.src
23+
const taskname = opts.taskname
24+
const prefixer = opts.autoprefixer
25+
const combinemq = opts.combinemq
26+
const minify = opts.minify
27+
const beautifier = opts.beautifier
28+
const plumb = opts.plumb
29+
const dest = opts.dest
30+
const size = opts.size
31+
const isProd = opts.production
3232

3333
gulp.task(taskname, done => gulp.src(src)
3434
.pipe(plumb ? $.plumber(config.plumberErrorHandler) : $.util.noop())
@@ -40,5 +40,5 @@ module.exports = (gulp, options) => {
4040
.pipe((minify && isProd) ? $.rename({ suffix: '.min' }) : $.util.noop())
4141
.pipe((!isProd && size) ? $.size({ title: `Gulp task ${taskname}`, gzip: false, showFiles: true }) : $.util.noop())
4242
.pipe(plumb ? $.plumber.stop() : $.util.noop())
43-
.pipe(gulp.dest(dest)));
44-
};
43+
.pipe(gulp.dest(dest)))
44+
}

package.json

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,11 @@
1111
"homepage": "https://github.com/LojasKD/gulp-tasks#readme",
1212
"bugs": "https://github.com/LojasKD/gulp-tasks/issues",
1313
"scripts": {
14+
"lint": "./node_modules/.bin/eslint ./**/*.js --quiet",
1415
"test": "./node_modules/.bin/tape test/*.spec.js | tap-spec",
1516
"publish": "npm publish --access=public",
16-
"prepublish": "npm test",
17-
"preversion": "npm test"
17+
"prepublish": "npm test && npm run lint",
18+
"preversion": "npm test && npm run lint"
1819
},
1920
"keywords": [
2021
"gulp",
@@ -47,7 +48,12 @@
4748
"require-dir": "^0.3.1"
4849
},
4950
"devDependencies": {
51+
"eslint": "^3.18.0",
52+
"eslint-config-standard": "^7.1.0",
53+
"eslint-plugin-promise": "^3.5.0",
54+
"eslint-plugin-standard": "^2.1.1",
5055
"gulp": "^3.9.1",
56+
"husky": "^0.13.3",
5157
"mkdirp": "^0.5.1",
5258
"tap-spec": "^4.1.1",
5359
"tape": "^4.6.3"

test/copy.spec.js

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

3-
var exec = require('child_process').exec;
4-
var tape = require('tape');
5-
var utils = require('./utils');
3+
var exec = require('child_process').exec
4+
var tape = require('tape')
5+
var utils = require('./utils')
66

77
tape('copy task', function (test) {
8-
test.plan(1);
8+
test.plan(1)
99
exec('gulp copy', { cwd: __dirname }, function () {
10-
var expected = utils.readFixture('static/logo.png');
11-
var actual = utils.readResult('logo.png');
12-
test.equal(expected, actual, 'Processes copy task: Files copied!');
13-
});
14-
});
10+
var expected = utils.readFixture('static/logo.png')
11+
var actual = utils.readResult('logo.png')
12+
test.equal(expected, actual, 'Processes copy task: Files copied!')
13+
})
14+
})

0 commit comments

Comments
 (0)