Disable prefer-optional-catch-binding rule globally (#128)

sindresorhus · Sep 24, 2020 · ef85dd0 · ef85dd0
1 parent e630220
commit ef85dd0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -84,6 +84,11 @@
 			}
 		}
 	},
+	"xo": {
+		"rules": {
+			"unicorn/prefer-optional-catch-binding": "off"
+		}
+	},
 	"nyc": {
 		"extension": [
 			".ts"

diff --git a/source/index.ts b/source/index.ts
@@ -137,7 +137,6 @@ class Conf<T extends Record<string, any> = Record<string, unknown>> implements I
 
 		try {
 			assert.deepEqual(fileStore, store);
-		// eslint-disable-next-line unicorn/prefer-optional-catch-binding
 		} catch (_) {
 			this.store = store;
 		}
@@ -369,10 +368,8 @@ class Conf<T extends Record<string, any> = Record<string, unknown>> implements I
 						const decipher = crypto.createDecipher(encryptionAlgorithm, this.#encryptionKey);
 						data = Buffer.concat([decipher.update(Buffer.from(data)), decipher.final()]).toString('utf8');
 					}
-				// eslint-disable-next-line unicorn/prefer-optional-catch-binding
 				} catch (_) { }
 			}
-		// eslint-disable-next-line unicorn/prefer-optional-catch-binding
 		} catch (_) {
 		}
 
@@ -402,7 +399,6 @@ class Conf<T extends Record<string, any> = Record<string, unknown>> implements I
 			try {
 				// TODO: Use `util.isDeepStrictEqual` when targeting Node.js 10
 				assert.deepEqual(newValue, oldValue);
-			// eslint-disable-next-line unicorn/prefer-optional-catch-binding
 			} catch (_) {
 				currentValue = newValue;
 				callback.call(this, newValue, oldValue);