Change eslint rule arrow-parens to always

tshemsedinov · tshemsedinov · commit 81048dcd93a5 · 2021-01-03T21:22:33.000+02:00
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -208,7 +208,7 @@
     ],
     "arrow-parens": [
       "error",
-      "as-needed"
+      "always"
     ],
     "arrow-body-style": [
       "error",
diff --git a/JavaScript/2-emulate-async-calls.js b/JavaScript/2-emulate-async-calls.js
@@ -13,7 +13,7 @@ const callbackCounter = (count, callback) => {
 
 // Emulate asynchronous calls
 
-const wrapAsync = fn => (...args) => setTimeout(
+const wrapAsync = (fn) => (...args) => setTimeout(
   () => fn(...args), Math.floor(Math.random() * 1000)
 );
 
diff --git a/JavaScript/3-back-to-order.js b/JavaScript/3-back-to-order.js
@@ -4,7 +4,7 @@
 
 // Emulate asynchronous calls
 
-const wrapAsync = fn => (...args) => setTimeout(
+const wrapAsync = (fn) => (...args) => setTimeout(
   () => fn(...args), Math.floor(Math.random() * 1000)
 );
 
diff --git a/JavaScript/4-step-by-step.js b/JavaScript/4-step-by-step.js
@@ -7,7 +7,7 @@ const data = {};
 
 // Emulate asynchronous calls
 
-const wrapAsync = fn => (...args) => setTimeout(
+const wrapAsync = (fn) => (...args) => setTimeout(
   () => fn(...args), Math.floor(Math.random() * 1000)
 );
 
diff --git a/JavaScript/5-chain.js b/JavaScript/5-chain.js
@@ -35,7 +35,7 @@ const chain = (prev = null) => {
 
 // Emulate asynchronous calls
 
-const wrapAsync = fn => (...args) => setTimeout(
+const wrapAsync = (fn) => (...args) => setTimeout(
   () => fn(...args), Math.floor(Math.random() * 1000)
 );
 
diff --git a/JavaScript/6-promise-sequential.js b/JavaScript/6-promise-sequential.js
@@ -2,39 +2,39 @@
 
 // Emulate asynchronous calls
 
-const wrapAsync = callback => setTimeout(
+const wrapAsync = (callback) => setTimeout(
   callback, Math.floor(Math.random() * 1000)
 );
 
 const isWeekend = () => !(new Date().getDay() % 6);
 
 // Asynchronous functions
 
-const readConfig = name => new Promise((resolve, reject) => {
+const readConfig = (name) => new Promise((resolve, reject) => {
   wrapAsync(() => {
     console.log('(1) config loaded');
     if (!isWeekend()) resolve({ name });
     else reject(new Error('Promises will resolve next working day'));
   });
 });
 
-const doQuery = statement => new Promise((resolve, reject) => {
+const doQuery = (statement) => new Promise((resolve, reject) => {
   wrapAsync(() => {
     console.log('(2) SQL query executed: ' + statement);
     if (!isWeekend()) resolve([{ name: 'Kiev' }, { name: 'Roma' }]);
     else reject(new Error('Promises will resolve next working day'));
   });
 });
 
-const httpGet = url => new Promise((resolve, reject) => {
+const httpGet = (url) => new Promise((resolve, reject) => {
   wrapAsync(() => {
     console.log('(3) Page retrieved: ' + url);
     if (!isWeekend()) resolve('<html>Some archaic web here</html>');
     else reject(new Error('Promises will resolve next working day'));
   });
 });
 
-const readFile = path => new Promise((resolve, reject) => {
+const readFile = (path) => new Promise((resolve, reject) => {
   wrapAsync(() => {
     console.log('(4) Readme file loaded: ' + path);
     if (!isWeekend()) resolve('file content');
@@ -48,10 +48,10 @@ Promise.resolve()
   .then(readConfig.bind(null, 'myConfig'))
   .then(doQuery.bind(null, 'select * from cities'))
   .then(httpGet.bind(null, 'http://kpi.ua'))
-  .catch(err => console.log('Reject reason (1): ' + err.message))
+  .catch((err) => console.log('Reject reason (1): ' + err.message))
   .then(readFile.bind(null, 'README.md'))
-  .catch(err => console.log('Reject reason (2): ' + err.message))
-  .then(data => {
+  .catch((err) => console.log('Reject reason (2): ' + err.message))
+  .then((data) => {
     console.log('Done');
     console.dir({ data });
   });
diff --git a/JavaScript/7-promise-all.js b/JavaScript/7-promise-all.js
@@ -2,34 +2,34 @@
 
 // Emulate asynchronous calls
 
-const wrapAsync = callback => setTimeout(
+const wrapAsync = (callback) => setTimeout(
   callback, Math.floor(Math.random() * 1000)
 );
 
 // Asynchronous functions
 
-const readConfig = name => new Promise(resolve => {
+const readConfig = (name) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(1) config loaded: ' + name);
     resolve({ name });
   });
 });
 
-const doQuery = statement => new Promise(resolve => {
+const doQuery = (statement) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(2) SQL query executed: ' + statement);
     resolve([ { name: 'Kiev' }, { name: 'Roma' } ]);
   });
 });
 
-const httpGet = url => new Promise(resolve => {
+const httpGet = (url) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(3) Page retrieved: ' + url);
     resolve('<html>Some archaic web here</html>');
   });
 });
 
-const readFile = path => new Promise(resolve => {
+const readFile = (path) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(4) Readme file loaded: ' + path);
     resolve('file content');
@@ -43,7 +43,7 @@ Promise.all([
   doQuery('select * from cities'),
   httpGet('http://kpi.ua'),
   readFile('README.md')
-]).then(data => {
+]).then((data) => {
   console.log('Done');
   console.dir({ data });
 });
diff --git a/JavaScript/8-promise-mixed.js b/JavaScript/8-promise-mixed.js
@@ -2,34 +2,34 @@
 
 // Emulate asynchronous calls
 
-const wrapAsync = callback => setTimeout(
+const wrapAsync = (callback) => setTimeout(
   callback, Math.floor(Math.random() * 1000)
 );
 
 // Asynchronous functions
 
-const readConfig = name => new Promise(resolve => {
+const readConfig = (name) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(1) config loaded: ' + name);
     resolve({ name });
   });
 });
 
-const doQuery = statement => new Promise(resolve => {
+const doQuery = (statement) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(2) SQL query executed: ' + statement);
     resolve([ { name: 'Kiev' }, { name: 'Roma' } ]);
   });
 });
 
-const httpGet = url => new Promise(resolve => {
+const httpGet = (url) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(3) Page retrieved: ' + url);
     resolve('<html>Some archaic web here</html>');
   });
 });
 
-const readFile = path => new Promise(resolve => {
+const readFile = (path) => new Promise((resolve) => {
   wrapAsync(() => {
     console.log('(4) Readme file loaded: ' + path);
     resolve('file content');
@@ -45,7 +45,7 @@ Promise.resolve()
     httpGet('http://kpi.ua')
   ]))
   .then(readFile.bind(null, 'README.md'))
-  .then(data => {
+  .then((data) => {
     console.log('Done');
     console.dir({ data });
   });
diff --git a/JavaScript/9-events.js b/JavaScript/9-events.js
@@ -5,28 +5,28 @@ const ee = new EventEmitter();
 
 // Emulate asynchronous calls
 
-const wrapAsync = fn => (...args) => setTimeout(
+const wrapAsync = (fn) => (...args) => setTimeout(
   () => fn(...args), Math.floor(Math.random() * 1000)
 );
 
 // Asynchronous functions
 
-const readConfig = wrapAsync(name => {
+const readConfig = wrapAsync((name) => {
   console.log('(1) config loaded');
   ee.emit('config', { name });
 });
 
-const doQuery = wrapAsync(statement => {
+const doQuery = wrapAsync((statement) => {
   console.log('(2) SQL query executed: ' + statement);
   ee.emit('query', [ { name: 'Kiev' }, { name: 'Roma' } ]);
 });
 
-const httpGet = wrapAsync(url => {
+const httpGet = wrapAsync((url) => {
   console.log('(3) Page retrieved: ' + url);
   ee.emit('page', '<html>Some archaic web here</html>');
 });
 
-const readFile = wrapAsync(path => {
+const readFile = wrapAsync((path) => {
   console.log('(4) Readme file loaded: ' + path);
   ee.emit('done', 'file content');
 });
diff --git a/JavaScript/a-to-async.js b/JavaScript/a-to-async.js
@@ -2,9 +2,9 @@
 
 // Sync function to async
 
-const last = arr => arr[arr.length - 1];
+const last = (arr) => arr[arr.length - 1];
 
-const asyncify = fn => (...args) => {
+const asyncify = (fn) => (...args) => {
   const callback = last(args);
   args.pop();
   process.nextTick(() => {
@@ -14,10 +14,10 @@ const asyncify = fn => (...args) => {
 
 // Functions
 
-const f1 = par => par;
-const f2 = par => par;
-const f3 = par => par;
-const f4 = par => par;
+const f1 = (par) => par;
+const f2 = (par) => par;
+const f3 = (par) => par;
+const f4 = (par) => par;
 
 // Usage
 
diff --git a/JavaScript/b-sync-to-promise.js b/JavaScript/b-sync-to-promise.js
@@ -2,7 +2,7 @@
 
 // Sync function to Promise
 
-const promisifySync = fn => (...args) => (
+const promisifySync = (fn) => (...args) => (
   new Promise((resolve, reject) => {
     const result = fn(...args);
     if (result instanceof Error) reject(result);
@@ -12,12 +12,12 @@ const promisifySync = fn => (...args) => (
 
 // Usage
 
-const f1 = par => par;
+const f1 = (par) => par;
 
 console.log('Sync execution:', f1('value'));
 
 const pf1 = promisifySync(f1);
 
 pf1('value')
-  .then(data => console.log('Promise execution:', data))
-  .catch(err => console.error(err));
+  .then((data) => console.log('Promise execution:', data))
+  .catch((err) => console.error(err));
diff --git a/JavaScript/c-promisify.js b/JavaScript/c-promisify.js
@@ -2,7 +2,7 @@
 
 // Async function to Promise
 
-const promisify = fn => (...args) => (
+const promisify = (fn) => (...args) => (
   new Promise((resolve, reject) => (
     fn(...args, (err, data) => (
       err ? reject(err) : resolve(data)
@@ -21,5 +21,5 @@ f1('value', (err, par) => (
 const pf1 = promisify(f1);
 
 pf1('value')
-  .then(data => console.log('Promise execution:', data))
-  .catch(err => console.error(err));
+  .then((data) => console.log('Promise execution:', data))
+  .catch((err) => console.error(err));
diff --git a/JavaScript/e-asyncjs-parallel.js b/JavaScript/e-asyncjs-parallel.js
@@ -5,9 +5,9 @@ const async = require('async');
 console.log('Parallel array of function');
 
 const arr = [
-  callback => callback(null, 'uno'),
-  callback => callback(null, 'due'),
-  callback => callback(null, 'tre')
+  (callback) => callback(null, 'uno'),
+  (callback) => callback(null, 'due'),
+  (callback) => callback(null, 'tre')
 ];
 
 async.parallel(
@@ -20,9 +20,9 @@ async.parallel(
 console.log('Parallel hash of function');
 
 const obj = {
-  key1: callback => callback(null, 'uno'),
-  key2: callback => callback(new Error('Oh, shit'), 'due'),
-  key3: callback => callback(null, 'tre')
+  key1: (callback) => callback(null, 'uno'),
+  key2: (callback) => callback(new Error('Oh, shit'), 'due'),
+  key3: (callback) => callback(null, 'tre')
 };
 
 async.parallel(
diff --git a/JavaScript/f-asyncjs-series.js b/JavaScript/f-asyncjs-series.js
@@ -5,9 +5,9 @@ const async = require('async');
 console.log('Parallel array of function');
 
 const arr = [
-  callback => callback(null, 'uno'),
-  callback => callback(null, 'due'),
-  callback => callback(null, 'tre')
+  (callback) => callback(null, 'uno'),
+  (callback) => callback(null, 'due'),
+  (callback) => callback(null, 'tre')
 ];
 
 async.series(
@@ -20,9 +20,9 @@ async.series(
 console.log('Parallel hash of function');
 
 const obj = {
-  key1: callback => callback(null, 'uno'),
-  key2: callback => callback(new Error('Oh, shit'), 'due'),
-  key3: callback => callback(null, 'tre')
+  key1: (callback) => callback(null, 'uno'),
+  key2: (callback) => callback(new Error('Oh, shit'), 'due'),
+  key3: (callback) => callback(null, 'tre')
 };
 
 async.series(
diff --git a/JavaScript/g-async-await.js b/JavaScript/g-async-await.js
@@ -2,31 +2,31 @@
 
 // Emulate Asynchronous calls
 
-const pause = () => new Promise(resolve =>
+const pause = () => new Promise((resolve) =>
   setTimeout(resolve, Math.floor(Math.random() * 1000))
 );
 
 // Asynchronous functions
 
-const readConfig = async name => {
+const readConfig = async (name) => {
   await pause();
   console.log('(1) config loaded');
   return { name };
 };
 
-const doQuery = async statement => {
+const doQuery = async (statement) => {
   await pause();
   console.log('(2) SQL query executed: ' + statement);
   return [{ name: 'Kiev' }, { name: 'Roma' }];
 };
 
-const httpGet = async url => {
+const httpGet = async (url) => {
   await pause();
   console.log('(3) Page retrieved: ' + url);
   return '<html>Some archaic web here</html>';
 };
 
-const readFile = async path => {
+const readFile = async (path) => {
   await pause();
   console.log('(4) Readme file loaded: ' + path);
   return 'file content';
