Skip to content

Commit

Permalink
Add more jscs checks and some jscs todos
Browse files Browse the repository at this point in the history
  • Loading branch information
c0bra committed Oct 8, 2014
1 parent 0f38644 commit 0c5fbc4
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 16 deletions.
3 changes: 2 additions & 1 deletion .jscs.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,6 @@
"disallowImplicitTypeConversion": ["string"],
"disallowMixedSpacesAndTabs": true,
"disallowTrailingComma": true,
"disallowYodaConditions": true
"disallowYodaConditions": true,
"disallowNewlineBeforeBlockStatements": true
}
22 changes: 7 additions & 15 deletions .jscs.json.todo
Original file line number Diff line number Diff line change
Expand Up @@ -4,24 +4,16 @@
// that correct the existing code base issues and make the new check pass.

{
"disallowLeftStickedOperators": ["?", "+", "-", "/", "*", "=", "==", "===", "!=", "!==", ">", ">=", "<", "<="],
"disallowRightStickedOperators": ["?", "+", "/", "*", ":", "=", "==", "===", "!=", "!==", ">", ">=", "<", "<="],
"disallowMultipleLineBreaks": true,

"disallowKeywordsOnNewLine": ["else"],
"requireKeywordsOnNewLine": ["else"],

"requireLineFeedAtFileEnd": true,
"validateJSDoc": {
"checkParamNames": true,
"requireParamTypes": true
},

// (c0bra): Dunno about this one... it doesn't allow blank lines with whitespace
"disallowTrailingWhitespace": true

// (c0bra): Not supported in jscs yet (currently 1.3.0)
// "requireSpaceBeforeBlockStatements": true

// This would prevent all-whitespace lines as well
"disallowTrailingWhitespace": tru

"requireSpaceBeforeBlockStatements": true
"requireSpaceBeforeObjectValues": true
"requireSpaceAfterLineComment": true
"validateLineBreaks": "LF"
"validateParameterSeparator": ", "
}

0 comments on commit 0c5fbc4

Please sign in to comment.