feat: add eslint-plugin-unicorn (#1936)

2 files changed, 17 insertions, 18 deletions

diff --git a/src/webview/lib/RecipeWebview.js b/src/webview/lib/RecipeWebview.js
index 157da7693..4085b925b 100644
--- a/src/webview/lib/RecipeWebview.js
+++ b/src/webview/lib/RecipeWebview.js
@@ -1,5 +1,5 @@
 import { ipcRenderer } from 'electron';
-import { exists, pathExistsSync, readFileSync } from 'fs-extra';
+import { pathExistsSync, readFileSync, existsSync } from 'fs-extra';
 
 const debug = require('debug')('Ferdi:Plugin:RecipeWebview');
 
@@ -62,12 +62,13 @@ class RecipeWebview {
    *                          be an absolute path to the file
    */
   injectCSS(...files) {
-    files.forEach(async (file) => {
+    // eslint-disable-next-line unicorn/no-array-for-each
+    files.forEach(file => {
       if (pathExistsSync(file)) {
         const styles = document.createElement('style');
         styles.innerHTML = readFileSync(file, 'utf8');
 
-        document.querySelector('head').appendChild(styles);
+        document.querySelector('head').append(styles);
 
         debug('Append styles', styles);
       }
@@ -75,14 +76,16 @@ class RecipeWebview {
   }
 
   injectJSUnsafe(...files) {
-    Promise.all(files.map(async (file) => {
-      if (await exists(file)) {
-        return readFileSync(file, 'utf8');
-      }
-      debug('Script not found', file);
-      return null;
-    })).then(async (scripts) => {
-      const scriptsFound = scripts.filter((script) => script !== null);
+    Promise.all(
+      files.map(file => {
+        if (existsSync(file)) {
+          return readFileSync(file, 'utf8');
+        }
+        debug('Script not found', file);
+        return null;
+      }),
+    ).then(scripts => {
+      const scriptsFound = scripts.filter(script => script !== null);
       if (scriptsFound.length > 0) {
         debug('Inject scripts to main world', scriptsFound);
         ipcRenderer.sendToHost('inject-js-unsafe', ...scriptsFound);
diff --git a/src/webview/lib/Userscript.js b/src/webview/lib/Userscript.js
index 2043d9fff..bed2b1ff8 100644
--- a/src/webview/lib/Userscript.js
+++ b/src/webview/lib/Userscript.js
@@ -28,7 +28,7 @@ export default class Userscript {
    *
    * @param {*} settings
    */
-  // eslint-disable-next-line
+  // eslint-disable-next-line camelcase
   internal_setSettings(settings) {
     // This is needed to get a clean JS object from the settings itself to provide better accessibility
     // Otherwise this will be a mobX instance
@@ -95,9 +95,7 @@ export default class Userscript {
    * @param {*} value
    */
   set(key, value) {
-    window.localStorage.setItem(
-      `ferdi-user-${key}`, JSON.stringify(value),
-    );
+    window.localStorage.setItem(`ferdi-user-${key}`, JSON.stringify(value));
   }
 
   /**
@@ -107,9 +105,7 @@ export default class Userscript {
    * @return Value of the key
    */
   get(key) {
-    return JSON.parse(window.localStorage.getItem(
-      `ferdi-user-${key}`,
-    ));
+    return JSON.parse(window.localStorage.getItem(`ferdi-user-${key}`));
   }
 
   /**