(re)enable no-extra-semi rule

.eslintrc.json

@@ -19,7 +19,7 @@
         "no-duplicate-case": "warn",
         "no-duplicate-imports": "warn",
         "no-eval": "warn",
-        "no-extra-semi": "off",
+        "no-extra-semi": "warn",
         "no-new-wrappers": "warn",
         "no-redeclare": "off",
         "no-sparse-arrays": "warn",

extensions/vscode-colorize-tests/src/colorizer.test.ts

@@ -56,7 +56,7 @@ function hasThemeChange(d: any, p: any) : boolean {
 		}
 	}
 	return false;
-};
+}
 
 suite('colorization', () => {
 	let extensionsFolder = normalize(join(__dirname, '../../'));

src/vs/workbench/test/electron-browser/api/extHostDiagnostics.test.ts

@@ -34,7 +34,7 @@ suite('ExtHostDiagnostics', () => {
 		assert.throws(() => collection.clear());
 		assert.throws(() => collection.delete(URI.parse('aa:bb')));
 		// tslint:disable-next-line:semicolon
-		assert.throws(() => collection.forEach(() => { ; }));
+		assert.throws(() => collection.forEach(() => { }));
 		assert.throws(() => collection.get(URI.parse('aa:bb')));
 		assert.throws(() => collection.has(URI.parse('aa:bb')));
 		assert.throws(() => collection.set(URI.parse('aa:bb'), []));