Fixes to formatting

DonJayamanne · DonJayamanne · commit 511346a21b5f · 2020-02-24T13:50:35.000-08:00
diff --git a/package.nls.json b/package.nls.json
@@ -444,7 +444,7 @@
     "DataScience.startingJupyterNotebook": "Starting Jupyter Notebook",
     "DataScience.registeringKernel": "Registering Kernel",
     "DataScience.trimmedOutput": "Output was trimmed for performance reasons.\nTo see the full output set the setting \"python.dataScience.textOutputLimit\" to 0.",
-    "DataScience.connectingToJupyterUri" : "Connecting to Jupyter server at {0}",
-    "DataScience.createdNewNotebook" : "{0}: Creating new notebook ",
-    "DataScience.createdNewKernel" : "{0}: Kernel started: {1}"
+    "DataScience.connectingToJupyterUri": "Connecting to Jupyter server at {0}",
+    "DataScience.createdNewNotebook": "{0}: Creating new notebook ",
+    "DataScience.createdNewKernel": "{0}: Kernel started: {1}"
 }
diff --git a/src/test/format/extension.onTypeFormat.test.ts b/src/test/format/extension.onTypeFormat.test.ts
@@ -331,7 +331,6 @@ suite('Try blocks with indentation of 4 spaces', () => {
     });
 });
 
-// tslint:disable-next-line: max-func-body-length
 suite('Try blocks with indentation of Tab', () => {
     suiteSetup(async () => {
         await initialize();
diff --git a/src/test/refactor/rename.test.ts b/src/test/refactor/rename.test.ts
@@ -31,7 +31,6 @@ type RenameResponse = {
     results: [{ diff: string }];
 };
 
-// tslint:disable-next-line: max-func-body-length
 suite('Refactor Rename', () => {
     const options: TextEditorOptions = {
         cursorStyle: TextEditorCursorStyle.Line,
diff --git a/src/test/testing/unittest/unittest.run.test.ts b/src/test/testing/unittest/unittest.run.test.ts
@@ -236,7 +236,6 @@ suite('Unit Tests - unittest - run with mocked process output', () => {
             assert.equal(results.summary.skipped, 1, 'skipped');
         });
 
-        // tslint:disable-next-line: max-func-body-length
         test(`Run Failed Tests [${cfg.startDirSwitch}, ${cfg.patternSwitch}]`, async () => {
             await updateSetting(
                 'testing.unittestArgs',
diff --git a/tsconfig.datascience-ui.json b/tsconfig.datascience-ui.json
@@ -6,17 +6,35 @@
         "importHelpers": true,
         "target": "es5",
         "outDir": "out",
-        "lib": ["es6", "dom"],
+        "lib": [
+            "es6",
+            "dom"
+        ],
         "jsx": "react",
         "sourceMap": true,
-        "rootDirs": ["node_modules/vsls", "src", "types"],
-        "paths": { "*": ["types/*"] },
+        "rootDirs": [
+            "node_modules/vsls",
+            "src",
+            "types"
+        ],
+        "paths": {
+            "*": [
+                "types/*"
+            ]
+        },
         "experimentalDecorators": true,
         "allowSyntheticDefaultImports": true,
         "noImplicitThis": false,
         "noUnusedLocals": true,
         "noUnusedParameters": false,
         "strict": true
     },
-    "exclude": [".vscode-test", ".vscode test", "src/test", "src/server", "src/client", "build"]
+    "exclude": [
+        ".vscode-test",
+        ".vscode test",
+        "src/test",
+        "src/server",
+        "src/client",
+        "build"
+    ]
 }
diff --git a/tsconfig.extension.json b/tsconfig.extension.json
@@ -4,13 +4,21 @@
         "module": "commonjs",
         "target": "es6",
         "outDir": "out",
-        "lib": ["es6"],
+        "lib": [
+            "es6"
+        ],
         "jsx": "react",
         "sourceMap": true,
         "rootDir": "src",
         "experimentalDecorators": true,
         "allowSyntheticDefaultImports": true,
         "noImplicitThis": false
     },
-    "exclude": ["node_modules", ".vscode-test", ".vscode test", "src/datascience-ui", "build"]
+    "exclude": [
+        "node_modules",
+        ".vscode-test",
+        ".vscode test",
+        "src/datascience-ui",
+        "build"
+    ]
 }
diff --git a/tsconfig.json b/tsconfig.json
@@ -1,11 +1,19 @@
 {
     "compilerOptions": {
         "baseUrl": ".",
-        "paths": { "*": ["types/*"] },
+        "paths": {
+            "*": [
+                "types/*"
+            ]
+        },
         "module": "commonjs",
         "target": "es2018",
         "outDir": "out",
-        "lib": ["es6", "es2018", "dom"],
+        "lib": [
+            "es6",
+            "es2018",
+            "dom"
+        ],
         "jsx": "react",
         "sourceMap": true,
         "rootDir": "src",
diff --git a/tslint.json b/tslint.json
@@ -1,9 +1,18 @@
 {
-    "rulesDirectory": ["./build/tslint-rules"],
+    "rulesDirectory": [
+        "./build/tslint-rules"
+    ],
     "linterOptions": {
-        "exclude": ["src/ipywidgets"]
+        "exclude": [
+            "src/ipywidgets"
+        ]
     },
-    "extends": ["tslint-eslint-rules", "tslint-microsoft-contrib", "tslint-plugin-prettier", "tslint-config-prettier"],
+    "extends": [
+        "tslint-eslint-rules",
+        "tslint-microsoft-contrib",
+        "tslint-plugin-prettier",
+        "tslint-config-prettier"
+    ],
     "rules": {
         "prettier": true,
         "messages-must-be-localized": true,
@@ -13,14 +22,23 @@
         "non-literal-fs-path": false,
         "newline-per-chained-call": false,
         "class-name": true,
-        "semicolon": [true, "always", "strict-bound-class-methods"],
+        "semicolon": [
+            true,
+            "always",
+            "strict-bound-class-methods"
+        ],
         "triple-equals": true,
         "no-relative-imports": false,
         "max-line-length": false,
         "typedef": false,
         "no-string-throw": true,
         "missing-jsdoc": false,
-        "one-line": [true, "check-catch", "check-finally", "check-else"],
+        "one-line": [
+            true,
+            "check-catch",
+            "check-finally",
+            "check-else"
+        ],
         "no-parameter-properties": false,
         "no-parameter-reassignment": false,
         "no-reserved-keywords": false,
@@ -29,7 +47,11 @@
         "align": false,
         "linebreak-style": false,
         "strict-boolean-expressions": false,
-        "await-promise": [true, "Thenable", "PromiseLike"],
+        "await-promise": [
+            true,
+            "Thenable",
+            "PromiseLike"
+        ],
         "completed-docs": false,
         "no-unsafe-any": false,
         "no-backbone-get-set-outside-model": false,
@@ -47,7 +69,9 @@
         "no-bitwise": false,
         "eofline": true,
         "switch-final-break": false,
-        "no-implicit-dependencies": ["vscode"],
+        "no-implicit-dependencies": [
+            "vscode"
+        ],
         "no-unnecessary-type-assertion": false,
         "no-submodule-imports": false,
         "no-redundant-jsdoc": false,
