ESlint no-unreachable, no-redeclare

.eslintrc

@@ -81,8 +81,8 @@
 		"o-eq-null": "off",
 		"no-cond-assign": "off",
 		"no-eq-null": "off",
-		"no-redeclare": "off",
+		// "no-redeclare": "off",
-		"no-unreachable": "off",
+		// "no-unreachable": "off",
 		// "no-nested-ternary": "off",
 		// "operator-linebreak": "off",
 		// "guard-for-in": "off",

src/meta/blacklist.js

@@ -99,18 +99,18 @@ Blacklist.validate = function (rules, callback) {
 		if (ip.isV4Format(rule)) {
 			ipv4.push(rule);
 			return true;
-		} else if (ip.isV6Format(rule)) {
+		}
+		if (ip.isV6Format(rule)) {
 			ipv6.push(rule);
 			return true;
-		} else if (isCidrSubnet.test(rule)) {
+		}
+		if (isCidrSubnet.test(rule)) {
 			cidr.push(rule);
 			return true;
-		} else {
-			invalid.push(rule);
-			return false;
 		}
 
-		return true;
+		invalid.push(rule);
+		return false;
 	});
 
 	callback(null, {

src/routes/index.js

@@ -132,7 +132,7 @@ module.exports = function (app, middleware, hotswapIds) {
 	userRoutes(router, middleware, controllers);
 	groupRoutes(router, middleware, controllers);
 
-	for (var x = 0; x < routers.length; x += 1) {
+	for (x = 0; x < routers.length; x += 1) {
 		app.use(relativePath, routers[x]);
 	}