Skip to content

Commit 5ea7700

Browse files
pdehaanphated
authored andcommitted
Scaffold: Use jscs & eslint with presets
1 parent eca3a23 commit 5ea7700

File tree

6 files changed

+37
-97
lines changed

6 files changed

+37
-97
lines changed

.eslintrc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
"extends": "gulp"
3+
}

.jscsrc

Lines changed: 1 addition & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,3 @@
11
{
2-
"disallowKeywords": ["with"],
3-
"disallowKeywordsOnNewLine": ["else"],
4-
"disallowMixedSpacesAndTabs": true,
5-
"disallowMultipleVarDecl": "exceptUndefined",
6-
"disallowNewlineBeforeBlockStatements": true,
7-
"disallowQuotedKeysInObjects": true,
8-
"disallowSpaceAfterObjectKeys": true,
9-
"disallowSpaceAfterPrefixUnaryOperators": true,
10-
"disallowSpacesInFunction": {
11-
"beforeOpeningRoundBrace": true
12-
},
13-
"disallowSpacesInsideParentheses": true,
14-
"disallowTrailingWhitespace": true,
15-
"maximumLineLength": null,
16-
"requireCamelCaseOrUpperCaseIdentifiers": true,
17-
"requireCapitalizedComments": true,
18-
"requireCapitalizedConstructors": true,
19-
"requireCurlyBraces": true,
20-
"requireSpaceAfterKeywords": [
21-
"if",
22-
"else",
23-
"for",
24-
"while",
25-
"do",
26-
"switch",
27-
"case",
28-
"return",
29-
"try",
30-
"catch",
31-
"typeof"
32-
],
33-
"requireSpaceAfterLineComment": true,
34-
"requireSpaceAfterBinaryOperators": true,
35-
"requireSpaceBeforeBinaryOperators": true,
36-
"requireSpaceBeforeBlockStatements": true,
37-
"requireSpaceBeforeObjectValues": true,
38-
"requireSpacesInFunction": {
39-
"beforeOpeningCurlyBrace": true
40-
},
41-
"requireTrailingComma": {
42-
"ignoreSingleLine": true
43-
},
44-
"requireBlocksOnNewline": true,
45-
"validateIndentation": 2,
46-
"validateLineBreaks": "LF",
47-
"validateQuoteMarks": "'"
2+
"preset": "gulp"
483
}

.jshintrc

Lines changed: 0 additions & 19 deletions
This file was deleted.

.travis.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,6 @@ node_js:
66
- "0.10"
77
after_script:
88
- npm run coveralls
9+
- npm run lint
910
git:
1011
depth: 10

package.json

Lines changed: 29 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -2,28 +2,10 @@
22
"name": "gulp-cli",
33
"description": "The streaming build system",
44
"version": "0.3.0",
5-
"homepage": "http://gulpjs.com",
6-
"repository": "gulpjs/gulp-cli",
75
"author": "Fractal <contact@wearefractal.com> (http://wearefractal.com/)",
8-
"tags": [
9-
"build",
10-
"stream",
11-
"system",
12-
"make",
13-
"tool",
14-
"asset",
15-
"pipeline"
16-
],
17-
"files": [
18-
"index.js",
19-
"lib",
20-
"bin",
21-
"completion"
22-
],
236
"bin": {
247
"gulp": "bin/gulp.js"
258
},
26-
"man": "gulp.1",
279
"dependencies": {
2810
"archy": "^1.0.0",
2911
"chalk": "^1.1.0",
@@ -39,26 +21,41 @@
3921
"devDependencies": {
4022
"code": "^1.2.1",
4123
"coveralls": "^2.7.0",
24+
"eslint": "^1.7.3",
25+
"eslint-config-gulp": "^2.0.0",
4226
"gulp": ">=3.8.10",
43-
"jscs": "^1.11.3",
44-
"jshint": "^2.5.0",
45-
"jshint-stylish": "^2.0.1",
27+
"jscs": "^2.3.5",
28+
"jscs-preset-gulp": "^1.0.0",
4629
"lab": "^5.1.0",
4730
"marked-man": "^0.1.3"
4831
},
49-
"scripts": {
50-
"prepublish": "marked-man --name gulp docs/CLI.md > gulp.1",
51-
"lint": "jshint lib index.js --reporter node_modules/jshint-stylish --exclude node_modules && jscs index.js lib bin test",
52-
"test": "npm run lint && lab -cv",
53-
"coveralls": "lab -r lcov | coveralls"
54-
},
5532
"engines": {
5633
"node": ">= 0.9"
5734
},
58-
"licenses": [
59-
{
60-
"type": "MIT",
61-
"url": "https://raw.githubusercontent.com/gulpjs/gulp/master/LICENSE"
62-
}
35+
"files": [
36+
"index.js",
37+
"lib",
38+
"bin",
39+
"completion"
40+
],
41+
"homepage": "http://gulpjs.com",
42+
"license": "MIT",
43+
"man": "gulp.1",
44+
"repository": "gulpjs/gulp-cli",
45+
"scripts": {
46+
"coveralls": "lab -r lcov | coveralls",
47+
"lint": "eslint . && jscs index.js bin/ lib/ test/",
48+
"prepublish": "marked-man --name gulp docs/CLI.md > gulp.1",
49+
"pretest": "npm run lint",
50+
"test": "lab -cv"
51+
},
52+
"tags": [
53+
"build",
54+
"stream",
55+
"system",
56+
"make",
57+
"tool",
58+
"asset",
59+
"pipeline"
6360
]
6461
}

test/.eslintrc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
"extends": "gulp/test"
3+
}

0 commit comments

Comments
 (0)