Commit 8ae00b8e authored by Philip Chimento's avatar Philip Chimento 🚮

CI: Adjust eslint and clang-format rules [skip eslint]

parent e52498f8
......@@ -12,6 +12,7 @@ DerivePointerAlignment: false
ForEachMacros: []
IndentWidth: 4
PointerAlignment: Left # Google style allows both, but clang-format doesn't
SpacesBeforeTrailingComments: 2
---
# We rely mostly on eslint for JavaScript linting, but this is a lax collection
# of rules that will auto-fix some JS things. We really should use eslint --fix
......@@ -20,8 +21,9 @@ PointerAlignment: Left # Google style allows both, but clang-format doesn't
Language: JavaScript
AlignAfterOpenBracket: DontAlign
AllowShortFunctionsOnASingleLine: false
JavaScriptQuotes: Single
JavaScriptQuotes: Leave # It's not smart enough to allow " to minimize escapes
PenaltyBreakBeforeFirstCallParameter: 1000000
PenaltyExcessCharacter: 1
SpacesBeforeTrailingComments: 2
SpacesInContainerLiterals: false
...
......@@ -4,6 +4,7 @@
},
"extends": "eslint:recommended",
"rules": {
"array-bracket-spacing": ["error", "never"],
"brace-style": "error",
"comma-spacing": [
"error",
......@@ -19,6 +20,13 @@
"MemberExpression": "off"
}
],
"key-spacing": [
"error",
{
"beforeColon": false,
"afterColon": true
}
],
"keyword-spacing": [
"error",
{
......@@ -36,6 +44,7 @@
"allowEmptyCatch": true
}
],
"nonblock-statement-body-position": ["error", "below"],
"quotes": [
"error",
"single",
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment