Skip to content

Commit 517530f

Browse files
committed
Enforce jQuery style using jscs
1 parent 6334769 commit 517530f

7 files changed

+536
-449
lines changed

.jscsrc

+82
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,82 @@
1+
{
2+
"requireCurlyBraces": [
3+
"if",
4+
"else",
5+
"for",
6+
"while",
7+
"do",
8+
"try",
9+
"catch"
10+
],
11+
"requireOperatorBeforeLineBreak": true,
12+
"requireParenthesesAroundIIFE": true,
13+
"requireMultipleVarDecl": "onevar",
14+
"requireCommaBeforeLineBreak": true,
15+
"requireCamelCaseOrUpperCaseIdentifiers": true,
16+
"requireDotNotation": true,
17+
"maximumLineLength": {
18+
"value": 100,
19+
"tabSize": 4,
20+
"allowUrlComments": true,
21+
"allowRegex": true
22+
},
23+
"validateQuoteMarks": { "mark": "\"", "escape": true },
24+
25+
"disallowMixedSpacesAndTabs": "smart",
26+
"disallowTrailingWhitespace": true,
27+
"disallowMultipleLineStrings": true,
28+
"disallowTrailingComma": true,
29+
30+
31+
"requireSpacesInFunctionExpression": {
32+
"beforeOpeningCurlyBrace": true
33+
},
34+
"requireSpaceAfterKeywords": [
35+
"if",
36+
"else",
37+
"for",
38+
"while",
39+
"do",
40+
"switch",
41+
"return",
42+
"try",
43+
"catch"
44+
],
45+
"requireSpacesInsideObjectBrackets": "all",
46+
"requireSpacesInsideArrayBrackets": "all",
47+
"requireSpacesInConditionalExpression": true,
48+
"requireSpaceAfterBinaryOperators": true,
49+
"requireLineFeedAtFileEnd": true,
50+
"requireSpaceBeforeBinaryOperators": [
51+
"=", "+=", "-=", "*=", "/=", "%=", "<<=", ">>=", ">>>=",
52+
"&=", "|=", "^=", "+=",
53+
54+
"+", "-", "*", "/", "%", "<<", ">>", ">>>", "&",
55+
"|", "^", "&&", "||", "===", "==", ">=",
56+
"<=", "<", ">", "!=", "!=="
57+
],
58+
"requireSpacesInAnonymousFunctionExpression": {
59+
"beforeOpeningCurlyBrace": true
60+
},
61+
"requireSpacesInNamedFunctionExpression": {
62+
"beforeOpeningCurlyBrace": true
63+
},
64+
"validateLineBreaks": "LF",
65+
66+
"disallowKeywords": [ "with" ],
67+
"disallowKeywordsOnNewLine": [ "else" ],
68+
"disallowSpacesInFunctionExpression": {
69+
"beforeOpeningRoundBrace": true
70+
},
71+
"disallowSpacesInNamedFunctionExpression": {
72+
"beforeOpeningRoundBrace": true
73+
},
74+
"disallowSpacesInAnonymousFunctionExpression": {
75+
"beforeOpeningRoundBrace": true
76+
},
77+
"disallowSpaceAfterObjectKeys": true,
78+
"disallowSpaceAfterPrefixUnaryOperators": true,
79+
"disallowSpaceBeforePostfixUnaryOperators": true,
80+
"disallowSpaceBeforeBinaryOperators": [ ",", ":" ],
81+
"disallowMultipleLineBreaks": true
82+
}

CHANGELOG.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# 1.7.1-0 / Unreleased
2-
*
2+
* Use jscs
33

44
# 1.7.0 / 2014-09-09
55
* [FEATURE] AMD support (topolm).

Gruntfile.coffee

+16-5
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,12 @@ module.exports = (grunt) ->
3434
# FTP upload the demo files (requires https://github.com/mar10/pyftpsync)
3535
cmd: "pyftpsync --progress upload . ftp://www.wwwendt.de/tech/demo/jquery-contextmenu --delete-unmatched --omit dist,node_modules,.*,_* -x"
3636

37+
jscs:
38+
src: ["jquery.ui-contextmenu.js", "test/tests.js"]
39+
options:
40+
config: ".jscsrc"
41+
force: true
42+
3743
jshint:
3844
files: ["jquery.ui-contextmenu.js", "test/tests.js"]
3945
options:
@@ -101,11 +107,16 @@ module.exports = (grunt) ->
101107
dest: "jquery.ui-contextmenu.min.js"
102108

103109
watch:
104-
jshint:
110+
dev:
105111
options:
106112
atBegin: true
107-
files: ["jquery.ui-contextmenu.js"]
108-
tasks: ["jshint"]
113+
files: ["jquery.ui-contextmenu.js", "test/tests.js"]
114+
tasks: ["jshint", "jscs"]
115+
# jshint:
116+
# options:
117+
# atBegin: true
118+
# files: ["jquery.ui-contextmenu.js"]
119+
# tasks: ["jshint"]
109120

110121
yabs:
111122
release:
@@ -133,8 +144,8 @@ module.exports = (grunt) ->
133144
grunt.loadNpmTasks key if key isnt "grunt" and key.indexOf("grunt") is 0
134145

135146
grunt.registerTask "server", ["connect:demo"]
136-
grunt.registerTask "dev", ["connect:dev", "watch:jshint"]
137-
grunt.registerTask "test", ["jshint", "qunit"]
147+
grunt.registerTask "dev", ["connect:dev", "watch:dev"]
148+
grunt.registerTask "test", ["jshint", "jscs", "qunit"]
138149
grunt.registerTask "sauce", ["connect:sauce", "saucelabs-qunit"]
139150
if parseInt(process.env.TRAVIS_PULL_REQUEST, 10) > 0
140151
# saucelab keys do not work on forks

0 commit comments

Comments
 (0)