Change eslint rule arrow-parens to always

tshemsedinov · tshemsedinov · commit 3a01c1788111 · 2021-01-05T22:40:16.000+02:00
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -213,7 +213,7 @@
     ],
     "arrow-parens": [
       "error",
-      "as-needed"
+      "always"
     ],
     "arrow-body-style": [
       "error",
diff --git a/JavaScript/3-functor.js b/JavaScript/3-functor.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const text = (s = '') => ({
-  line: a => text(`${s}\n${a}`),
+  line: (a) => text(`${s}\n${a}`),
   toString: () => s
 });
 
diff --git a/JavaScript/4-mutable.js b/JavaScript/4-mutable.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const text = (s = '', o = {
-  line: a => (s += '\n' + a, o),
+  line: (a) => (s += '\n' + a, o),
   toString: () => s
 }) => o;
 
diff --git a/JavaScript/5-iterator.js b/JavaScript/5-iterator.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const text = (s = '', o = {
-  line: a => (s +=  '\n' + a, o),
+  line: (a) => (s +=  '\n' + a, o),
   [Symbol.iterator]: () => ({
     next() {
       const res = { value: s, done: this.finished };
diff --git a/JavaScript/6-promise.js b/JavaScript/6-promise.js
@@ -15,13 +15,13 @@ class ArrayChain {
 
   fetch(fn) {
     return (this
-      .then(data => fn(null, data))
-      .catch(err => fn(err))
+      .then((data) => fn(null, data))
+      .catch((err) => fn(err))
     );
   }
 
   _chain(performer, fn, initial) {
-    this._promise = this._promise.then(array => (
+    this._promise = this._promise.then((array) => (
       new Promise((resolve, reject) => (
         performer(array, fn, (err, result) => (
           (err ? reject(err) : resolve(result))
@@ -62,5 +62,5 @@ class ArrayChain {
 }
 
 module.exports = {
-  for: array => new ArrayChain(array)
+  for: (array) => new ArrayChain(array)
 };
diff --git a/JavaScript/7-functor.js b/JavaScript/7-functor.js
@@ -17,14 +17,14 @@ const arrayChain = (array, prev = null) => {
 
   if (!prev) process.nextTick(() => self(null, array));
 
-  self.then = fn => (done = fn, self);
-  self.catch = fn => (fail = fn, self);
-  self.fetch = fn => (self
-    .then(data => fn(null, data))
-    .catch(err => fn(err))
+  self.then = (fn) => (done = fn, self);
+  self.catch = (fn) => (fail = fn, self);
+  self.fetch = (fn) => (self
+    .then((data) => fn(null, data))
+    .catch((err) => fn(err))
   );
 
-  const chain = performer => (fn, initial) => {
+  const chain = (performer) => (fn, initial) => {
     const res = arrayChain(null, self);
     next = () => performer(array, fn, res, initial);
     return res;
diff --git a/JavaScript/8-prototype.js b/JavaScript/8-prototype.js
@@ -2,14 +2,14 @@
 
 const metasync = require('metasync');
 
-const async = op => {
+const async = (op) => {
   switch (op) {
-    case 'map': return metasync.map;
-    case 'filter': return metasync.filter;
-    case 'reduce': return metasync.reduce;
-    case 'each': return metasync.each;
-    case 'series': return metasync.series;
-    case 'find': return metasync.find;
+  case 'map': return metasync.map;
+  case 'filter': return metasync.filter;
+  case 'reduce': return metasync.reduce;
+  case 'each': return metasync.each;
+  case 'series': return metasync.series;
+  case 'find': return metasync.find;
   }
 };
 
@@ -54,7 +54,7 @@ ArrayChain.prototype.catch = function(fn) {
 };
 
 ArrayChain.prototype.fetch = function(fn) {
-  this.chain.push({ op: 'then', fn: res => fn(null, res) });
+  this.chain.push({ op: 'then', fn: (res) => fn(null, res) });
   this.chain.push({ op: 'catch', fn });
   this.execute();
   return this;
@@ -91,5 +91,5 @@ ArrayChain.prototype.find = function(fn) {
 };
 
 module.exports = {
-  for: array => new ArrayChain(array)
+  for: (array) => new ArrayChain(array)
 };
diff --git a/JavaScript/9-build.js b/JavaScript/9-build.js
@@ -2,14 +2,14 @@
 
 const metasync = require('metasync');
 
-const async = op => {
+const async = (op) => {
   switch (op) {
-    case 'map': return metasync.map;
-    case 'filter': return metasync.filter;
-    case 'reduce': return metasync.reduce;
-    case 'each': return metasync.each;
-    case 'series': return metasync.series;
-    case 'find': return metasync.find;
+  case 'map': return metasync.map;
+  case 'filter': return metasync.filter;
+  case 'reduce': return metasync.reduce;
+  case 'each': return metasync.each;
+  case 'series': return metasync.series;
+  case 'find': return metasync.find;
   }
 };
 
@@ -44,20 +44,20 @@ ArrayChain.prototype.execute = function(err) {
 };
 
 ['then', 'catch', 'map', 'filter', 'reduce', 'each', 'series', 'find']
-  .map(op => {
+  .map((op) => {
     ArrayChain.prototype[op] = function(fn) {
       this.chain.push({ op, fn });
       return this;
     };
   });
 
 ArrayChain.prototype.fetch = function(fn) {
-  this.chain.push({ op: 'then', fn: res => fn(null, res) });
+  this.chain.push({ op: 'then', fn: (res) => fn(null, res) });
   this.chain.push({ op: 'catch', fn });
   this.execute();
   return this;
 };
 
 module.exports = {
-  for: array => new ArrayChain(array)
+  for: (array) => new ArrayChain(array)
 };
diff --git a/JavaScript/a-test.js b/JavaScript/a-test.js
@@ -7,8 +7,8 @@ module.exports = (chaining, done) => {
 
   chaining
     .for([1, 2, 3, 4])
-    .filter(item => item % 2 === 0)
-    .map(item => item * 2)
+    .filter((item) => item % 2 === 0)
+    .map((item) => item * 2)
     .reduce((a, b) => a + b)
     .fetch((err, result) => {
       if (err) throw err;
diff --git a/JavaScript/b-compare.js b/JavaScript/b-compare.js
@@ -2,7 +2,7 @@
 
 const implementations = [
   '6-promise', '7-functor', '8-prototype', '9-build'
-].map(name => `./${name}.js`).map(require);
+].map((name) => `./${name}.js`).map(require);
 
 const test = require('./a-test.js');
 
diff --git a/JavaScript/d-build.js b/JavaScript/d-build.js
@@ -4,4 +4,4 @@ const benchmark = require('./c-benchmark');
 const chainPrototype = require('./9-build');
 const test = require('./a-test');
 
-benchmark.do(40000, 'ProtoBuild', done => test(chainPrototype, done));
+benchmark.do(40000, 'ProtoBuild', (done) => test(chainPrototype, done));
diff --git a/JavaScript/d-functor.js b/JavaScript/d-functor.js
@@ -4,4 +4,4 @@ const benchmark = require('./c-benchmark');
 const chainFunctor = require('./7-functor');
 const test = require('./a-test');
 
-benchmark.do(40000, 'Functor', done => test(chainFunctor, done));
+benchmark.do(40000, 'Functor', (done) => test(chainFunctor, done));
diff --git a/JavaScript/d-promise.js b/JavaScript/d-promise.js
@@ -4,4 +4,4 @@ const benchmark = require('./c-benchmark');
 const chainPromise = require('./6-promise');
 const test = require('./a-test');
 
-benchmark.do(40000, 'Promise', done => test(chainPromise, done));
+benchmark.do(40000, 'Promise', (done) => test(chainPromise, done));
diff --git a/JavaScript/d-prototype.js b/JavaScript/d-prototype.js
@@ -4,4 +4,4 @@ const benchmark = require('./c-benchmark');
 const chainPrototype = require('./8-prototype');
 const test = require('./a-test');
 
-benchmark.do(40000, 'Prototype', done => test(chainPrototype, done));
+benchmark.do(40000, 'Prototype', (done) => test(chainPrototype, done));
