fix: update "space-before-function-paren" linting rule and its description. Closes #3

.eslintrc.json

@@ -7,8 +7,12 @@
     "mocha": true
   },
   "extends": "eslint:recommended",
+  "globals": {
+    "Atomics": "readonly",
+    "SharedArrayBuffer": "readonly"
+  },
   "parserOptions": {
-    "sourceType": "module"
+    "ecmaVersion": 2018
   },
   "plugins": [
     "json"
@@ -21,8 +25,12 @@
     "quotes": ["error","single"],
     "semi": "error",
     "space-before-blocks": "error",
-    "space-before-function-paren": "error",
+    "space-before-function-paren": ["error", {
+      "anonymous": "always",
+      "named": "never",
+      "asyncArrow": "always" }
+    ],
     "no-undef": "off",
     "no-unused-vars": "off"
   }
-}
+}

README.md

@@ -74,18 +74,46 @@ function hello () {
   console.log('hello');
 }
 ```
-- Put a space between a function name and its arguments when you declare it (not when you call it).
+- Put a space before the parentheses when you write anonymous functions.
 ```js
 // Bad
-function log(str) {
+const callback = function(str) {
   console.log(str);
 }
 
 // Good
-function log (str) {
+const callback = function (str) {
   console.log(str);
 }
 ```
+- Don't put a space before the parentheses when you write named functions or call the function.
+```js
+// Bad
+function callback (str) => {
+  console.log(str);
+}
+
+callback ('hello, world!');
+
+// Good
+function callback(str) => {
+  console.log(str);
+}
+
+callback('hello, world!');
+```
+- Put a space after `async` keyword for asynchronous functions.
+```js
+// Bad
+const callback = async(str) => {
+  await Promise.resolve(str);
+}
+
+// Good
+const callback = async (str) => {
+  await Promise.resolve(str);
+}
+```
 - Put a space before blocks, and around keywords.
 ```js
 // Bad

gulpfile.js

@@ -1,10 +1,9 @@
-
 const gulp = require('gulp');
 const $ = require('gulp-load-plugins')();
 
 // Utility to ignore Node modules and Bower components
 // when generating the glob patterns array for gulp.src()
-function addDefSrcIgnore (srcArr) {
+function addDefSrcIgnore(srcArr) {
   return srcArr.concat([
     '!node_modules{,/**}',
     '!private{,/**}',
@@ -15,7 +14,7 @@ function addDefSrcIgnore (srcArr) {
 }
 
 // JavaScript and JSON linter
-function lintJs () {
+function lintJs() {
   return gulp.src(addDefSrcIgnore(['**/*.js', '**/*.json']), {dot: true})
     .pipe($.eslint({dotfiles: true}))
     .pipe($.eslint.format())

linters/.eslintrc.json

@@ -7,15 +7,30 @@
     "mocha": true
   },
   "extends": "eslint:recommended",
-  "parserOptions": {"sourceType": "module"},
-  "plugins": ["json"],
+  "globals": {
+    "Atomics": "readonly",
+    "SharedArrayBuffer": "readonly"
+  },
+  "parserOptions": {
+    "ecmaVersion": 2018
+  },
+  "plugins": [
+    "json"
+  ],
   "rules": {
-    "indent": ["error", 2],
+    "indent": ["error",2],
     "keyword-spacing": "error",
     "linebreak-style": "error",
-    "quotes": ["error","single"],
+    "no-extra-boolean-cast": "off",
+    "quotes": ["error", "single"],
     "semi": "error",
     "space-before-blocks": "error",
-    "space-before-function-paren": "error"
+    "space-before-function-paren": ["error", {
+      "anonymous": "always",
+      "named": "never",
+      "asyncArrow": "always" }
+    ],
+    "no-undef": "off",
+    "no-unused-vars": "off"
   }
-}
+}