Remove console logging

SinclaM · Jun 8, 2022 · e4cfd98 · e4cfd98
1 parent 58642cf
commit e4cfd98
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 66 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -1,62 +1,43 @@
 {
-  "env": {
-    "browser": true,
-    "es2021": true
-  },
-  "extends": [
-    "eslint:recommended",
-    "plugin:@typescript-eslint/recommended",
-    "plugin:import/errors",
-    "plugin:import/warnings"
-  ],
-  "parser": "@typescript-eslint/parser",
-  "parserOptions": {
-    "ecmaVersion": 12,
-    "sourceType": "module"
-  },
-  "plugins": [
-    "@typescript-eslint",
-    "import"
-  ],
-  "rules": {
-    "linebreak-style": [
-      "error",
-      "unix"
+    "env": {
+        "browser": true,
+        "es2021": true
+    },
+    "extends": [
+        "eslint:recommended",
+        "plugin:@typescript-eslint/recommended",
+        "plugin:import/errors",
+        "plugin:import/warnings"
     ],
-    "quotes": [
-      "error",
-      "single"
-    ],
-    "semi": [
-      "error",
-      "always"
-    ],
-    "eol-last": [
-      "error",
-      "always"
-    ],
-    "import/newline-after-import": [
-      "error",
-      {
-        "count": 1
-      }
-    ],
-    "@typescript-eslint/explicit-function-return-type": "off",
-    "@typescript-eslint/explicit-module-boundary-types": "off",
-    "@typescript-eslint/ban-ts-comment": "off",
-    "@typescript-eslint/no-inferrable-types": "off",
-    "@typescript-eslint/no-unused-vars": "error"
-  },
-  "settings": {
-    "import/resolver": {
-      "node": {
-        "extensions": [
-          ".js",
-          ".jsx",
-          ".ts",
-          ".tsx"
-        ]
-      }
+    "parser": "@typescript-eslint/parser",
+    "parserOptions": {
+        "ecmaVersion": 12,
+        "sourceType": "module"
+    },
+    "plugins": ["@typescript-eslint", "import"],
+    "rules": {
+        "linebreak-style": ["error", "unix"],
+        "quotes": ["error", "single"],
+        "semi": ["error", "always"],
+        "eol-last": ["error", "always"],
+        "import/newline-after-import": [
+            "error",
+            {
+                "count": 1
+            }
+        ],
+        "@typescript-eslint/explicit-function-return-type": "off",
+        "@typescript-eslint/explicit-module-boundary-types": "off",
+        "no-console": "error",
+        "@typescript-eslint/ban-ts-comment": "off",
+        "@typescript-eslint/no-inferrable-types": "off",
+        "@typescript-eslint/no-unused-vars": "error"
+    },
+    "settings": {
+        "import/resolver": {
+            "node": {
+                "extensions": [".js", ".jsx", ".ts", ".tsx"]
+            }
+        }
     }
-  }
 }
diff --git a/src/popup.ts b/src/popup.ts
@@ -20,14 +20,14 @@ setToDesmosDefault.onclick = function () {
 };
 
 // Log changes to storage for testing
-browser.storage.onChanged.addListener(function (changes, namespace) {
-    for (const [key, { oldValue, newValue }] of Object.entries(changes)) {
-        console.log(
-            `Storage key '${key}' in namespace '${namespace}' changed.`,
-            `Old value was '${oldValue}', new value is '${newValue}'.`
-        );
-    }
-});
+// browser.storage.onChanged.addListener(function (changes, namespace) {
+//     for (const [key, { oldValue, newValue }] of Object.entries(changes)) {
+//         console.log(
+//             `Storage key '${key}' in namespace '${namespace}' changed.`,
+//             `Old value was '${oldValue}', new value is '${newValue}'.`
+//         );
+//     }
+// });
 
 // Set many sliders at once. Used when the user presses a reset-to-default button.
 // toSet is an array of all the autoCommands to be set.