mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-09-24 18:58:42 +00:00
cf7f2b141d
Let's experiment with this via our local Gruntfile. If it works fine we can install it in Jenkins (similar to node-csslint). Verify through $ npm install && npm test; Fixed all outstanding violations. Also: * Added syntaxhighight to ignore. * Added imetests (which contain unformatted JSON) to ignore. * In ve.dm.ModelRegistry#matchTypeRegExps, removed redundant !! cast from the [+!!withFunc] statement which was hitting a bug in node-jscs. All callers to this local private function pass a literal boolean true/false so no need to cast it. * Removed "/* key .. , value */" from ve.setProp, though this wasn't caught by node-jscs, found it when searching for " , ". * Made npm.devDependencies fixed instead of using tilde-ranges. This too often leads to strange bugs or sudden changes. Fixed them at the version they were currently ranging to. Bug: 54218 Change-Id: Ib2630806f3946874c8b01e58cf171df83a28da29
24 lines
1.1 KiB
JSON
24 lines
1.1 KiB
JSON
{
|
|
"requireCurlyBraces": ["if", "else", "for", "while", "do"],
|
|
"requireSpaceAfterKeywords": ["if", "else", "for", "while", "do", "switch", "return", "function"],
|
|
"requireSpacesInFunctionExpression": {
|
|
"beforeOpeningCurlyBrace": true
|
|
},
|
|
"requireMultipleVarDecl": true,
|
|
"requireSpacesInsideObjectBrackets": "all",
|
|
"disallowSpaceAfterObjectKeys": true,
|
|
"disallowLeftStickedOperators": ["?", "+", "/", "*", "-", "=", "==", "===", "!=", "!==", ">", ">=", "<", "<="],
|
|
"disallowRightStickedOperators": ["?", "/", "*", ":", "=", "==", "===", "!=", "!==", ">", ">=", "<", "<="],
|
|
"requireRightStickedOperators": ["!"],
|
|
"requireLeftStickedOperators": [","],
|
|
"disallowSpaceAfterPrefixUnaryOperators": ["++", "--", "+", "-", "~"],
|
|
"disallowSpaceBeforePostfixUnaryOperators": ["++", "--"],
|
|
"requireSpaceBeforeBinaryOperators": ["+", "-", "/", "*", "=", "==", "===", "!=", "!=="],
|
|
"requireSpaceAfterBinaryOperators": ["+", "-", "/", "*", "=", "==", "===", "!=", "!=="],
|
|
"disallowKeywords": [ "with" ],
|
|
"disallowMultipleLineBreaks": true,
|
|
"validateLineBreaks": "LF",
|
|
"disallowKeywordsOnNewLine": ["else"],
|
|
"requireLineFeedAtFileEnd": true
|
|
}
|