Enabling a few more tslint rules in the codebase

src/vs/platform/credentials/test/node/keytar.test.ts

@@ -26,6 +26,7 @@ suite('Keytar', () => {
 				try {
 					await keytar.deletePassword(name, 'foo');
 				} finally {
+					// tslint:disable-next-line: no-unsafe-finally
 					throw err;
 				}
 			}

src/vs/workbench/api/node/extHostTypeConverters.ts

@@ -1008,8 +1008,6 @@ export namespace LogLevel {
 				return _MainLogLevel.Error;
 			case types.LogLevel.Critical:
 				return _MainLogLevel.Critical;
-			case types.LogLevel.Critical:
-				return _MainLogLevel.Critical;
 			case types.LogLevel.Off:
 				return _MainLogLevel.Off;
 		}
@@ -1031,8 +1029,6 @@ export namespace LogLevel {
 				return types.LogLevel.Error;
 			case _MainLogLevel.Critical:
 				return types.LogLevel.Critical;
-			case _MainLogLevel.Critical:
-				return types.LogLevel.Critical;
 			case _MainLogLevel.Off:
 				return types.LogLevel.Off;
 		}

tslint.json

@@ -3,11 +3,22 @@
 		"build/lib/tslint"
 	],
 	"rules": {
-		"no-string-throw": true,
-		"no-unused-expression": true,
+		"no-construct": true,
+		"no-duplicate-super": true,
+		"no-duplicate-switch-case": true,
 		"no-duplicate-variable": true,
+		"no-eval": true,
+		"no-sparse-arrays": true,
+		"no-string-throw": true,
+		"no-unbound-method": [
+			true,
+			"ignore-static"
+		],
+		"no-unsafe-finally": true,
+		"no-unused-expression": true,
 		"curly": true,
 		"class-name": true,
+		"label-position": true,
 		"semicolon": [
 			true,
 			"always"