tools: require function declarations

Except for arrow functions, require function declarations instead of function expressions via linting. This is the predominant style in our code base (77 instances of expressions to 2344 instances of declarations). PR-URL: #12711 Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]>
nodejs · May 1, 2017 · aea7269 · aea7269
1 parent a180259
commit aea7269
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/.eslintrc.yaml b/.eslintrc.yaml
@@ -97,6 +97,7 @@ rules:
   eol-last: 2
   func-call-spacing: 2
   func-name-matching: 2
+  func-style: [2, declaration, {allowArrowFunctions: true}]
   indent: [2, 2, {ArrayExpression: first,
                   CallExpression: {arguments: first},
                   MemberExpression: 1,