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 Backport-PR-URL: #13774 Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]>
nodejs · Jul 11, 2017 · 3d564a4 · 3d564a4
1 parent 7e6a956
commit 3d564a4
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/.eslintrc.yaml b/.eslintrc.yaml
@@ -92,6 +92,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,