eslint: Make spaces in functions declarations matter

This seems to be popular and is what the `standard` linter
many have used seems to do.

Test Plan:
  - Things should work like these examples indicate:
    http://eslint.org/docs/rules/space-before-function-paren#always

Change-Id: I02a06cd1c89b2bd44b6a96f5f9bfcb9874b95be8
Reviewed-on: https://gerrit.instructure.com/94904
Reviewed-by: Jon Willesen <jonw+gerrit@instructure.com>
Tested-by: Jenkins
Product-Review: Clay Diffrient <cdiffrient@instructure.com>
QA-Review: Clay Diffrient <cdiffrient@instructure.com>

.eslintrc.json

@@ -26,6 +26,7 @@
     "no-continue": ["off"],
     "no-plusplus": ["off"],
     "semi": ["off"],
+    "space-before-function-paren": ["error", "always"],
     "import/no-amd": ["off"]
   }
 }