Remove // tslint:disable comments, add TODOs to tslint.json (#21292)

types/d3/v3/index.d.ts

@@ -3,8 +3,6 @@
 // Definitions by: Alex Ford <https://github.com/gustavderdrache>, Boris Yankov <https://github.com/borisyankov>
 // Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
 
-/* tslint:disable */
-
 // Latest patch version of module validated against: 3.5.17
 
 export = d3;

types/firebird/index.d.ts

@@ -445,7 +445,6 @@ declare module 'firebird' {
      */
     class Stream extends stream.Stream {
         constructor(blob: FBBlob);
-        /* tslint:disable */
 
         /* NodeJS.ReadStream */
         readable: boolean;
@@ -462,7 +461,6 @@ declare module 'firebird' {
         end(str: string, encoding?: string, cb?: Function): void;
         destroy(error?: Error): void;
 
-        /* tslint:enable */
         check_destroyed(): void;
     }
 }

types/firebird/tslint.json

@@ -2,8 +2,11 @@
     "extends": "dtslint/dt.json",
     "rules": {
         // TODO
+        "ban-types": false,
         "no-boolean-literal-compare": false,
         "no-declare-current-package": false,
-        "no-unnecessary-generics": false
+        "no-single-declare-module": false,
+        "no-unnecessary-generics": false,
+        "unified-signatures": false
     }
 }

types/oracledb/index.d.ts

@@ -481,7 +481,6 @@ declare namespace oracledb {
 		terminate(): IPromise<void>;
 	}
 
-	/* tslint:disable no-unused-variable */
 	const DEFAULT: number;
 	/** Metadata return type */
 	const DB_TYPE_VARCHAR: number;
@@ -539,7 +538,6 @@ declare namespace oracledb {
 	 */
 	function newLob(iLob: ILob): Lob;
 
-	/* tslint:disable no-var-keyword */
 	/** Default transaction behaviour of auto commit for each statement. */
 	var autoCommit: boolean;
 	/**

types/selenium-webdriver/v2/chrome.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 import * as remote from './remote';
 

types/selenium-webdriver/v2/edge.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 import * as remote from './remote';
 

types/selenium-webdriver/v2/executors.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 
 /**

types/selenium-webdriver/v2/firefox.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 import * as remote from './remote';
 

types/selenium-webdriver/v2/http.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 
 /**

types/selenium-webdriver/v2/ie.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 
 /**

types/selenium-webdriver/v2/index.d.ts

@@ -3,7 +3,6 @@
 // Definitions by: Bill Armstrong <https://github.com/BillArmstrong>, Yuki Kokubun <https://github.com/Kuniwak>, Craig Nishina <https://github.com/cnishina>
 // Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
 // TypeScript Version: 2.1
-/* tslint:disable */
 
 import * as chrome from './chrome';
 import * as edge from './edge';

types/selenium-webdriver/v2/opera.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 import * as remote from './remote';
 

types/selenium-webdriver/v2/remote.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 
 /**

types/selenium-webdriver/v2/safari.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 import * as webdriver from './index';
 
 export class Server { }

types/selenium-webdriver/v2/testing.d.ts

@@ -1,4 +1,3 @@
-/* tslint:disable */
 /**
 * Registers a new test suite.
 * @param name The suite name.

types/selenium-webdriver/v2/tslint.json

@@ -1,6 +1,23 @@
 {
     "extends": "dtslint/dt.json",
     "rules": {
-        "dt-header": false
+        // TODOs
+        "ban-types": false,
+        "callable-types": false,
+        "dt-header": false,
+        "interface-name": false,
+        "jsdoc-format": false,
+        "no-any-union": false,
+        "no-consecutive-blank-lines": false,
+        "no-empty-interface": false,
+        "no-padding": false,
+        "no-unnecessary-class": false,
+        "no-unnecessary-generics": false,
+        "no-redundant-jsdoc": false,
+        "no-redundant-jsdoc-2": false,
+        "no-var-keyword": false,
+        "semicolon": false,
+        "strict-export-declare-modifiers": false,
+        "unified-signatures": false
     }
 }

types/stack-utils/index.d.ts

@@ -6,7 +6,6 @@
 
 export = StackUtils;
 
-// tslint:disable ban-types
 declare class StackUtils {
     static nodeInternals(): RegExp[];
     constructor(options?: StackUtils.Options);

types/stack-utils/tslint.json

@@ -1 +1,7 @@
-{ "extends": "dtslint/dt.json" }
+{
+    "extends": "dtslint/dt.json",
+    "rules": {
+        // TODO
+        "ban-types": false
+    }
+}