Removes references to tslint (#619)

* Removes references to tslint in contract-wrappers
which are obsoleted since https://github.com/0xProject/protocol/pull/584

* Remove tslint references in contracts/utils and test utils
obsoleted by https://github.com/0xProject/protocol/pull/589

* Remove tslint references in contracts/zeroex and test utils
obsoleted by https://github.com/0xProject/protocol/pull/587

* Remove other obsoleted tslint references

* Update contributing guidelines with eslint

* Fix prettier errors

CONTRIBUTING.md

@@ -59,16 +59,14 @@ We strongly recommend you use the [VSCode](https://code.visualstudio.com/) text
 
 #### Linter
 
-We use [TSLint](https://palantir.github.io/tslint/) with [custom configs](https://github.com/0xProject/0x-tools/tree/development/packages/tslint-config) to keep our code-style consistent.
+We use [ESLint](https://eslint.org/docs/latest/) to keep our code-style consistent.
 
-Use `yarn:lint` to lint the entire monorepo, and `PKG={PACKAGE_NAME} yarn lint` to lint a specific package.
+Use `yarn lint` to lint the entire monorepo, and `PKG={PACKAGE_NAME} yarn lint` to lint a specific package.
-
-If you want to change a rule, or add a custom rule, please make these changes to our [tslint-config](https://github.com/0xProject/0x-tools/tree/development/packages/tslint-config) package. All other packages have it as a dependency.
 
 Integrate it into your text editor:
 
--   VSCode: [vscode-tslint](https://marketplace.visualstudio.com/items?itemName=eg2.tslint)
+-   VSCode: [ESLint](https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint)
--   Atom: [linter-tslint](https://atom.io/packages/linter-tslint)
+-   Atom: [ESLint](https://atom.io/packages/eslint)
 
 #### Auto-formatter
 

contracts/test-utils/src/log_decoder.ts

@@ -35,7 +35,6 @@ export class LogDecoder {
     }
     public decodeLogOrThrow<ArgsType extends DecodedLogArgs>(log: LogEntry): LogWithDecodedArgs<ArgsType> | RawLog {
         const logWithDecodedArgsOrLog = this._abiDecoder.tryToDecodeLogOrNoop(log);
-        // tslint:disable-next-line:no-unnecessary-type-assertion
         if ((logWithDecodedArgsOrLog as LogWithDecodedArgs<ArgsType>).args === undefined) {
             throw new Error(`Unable to decode log: ${JSON.stringify(log)}`);
         }

contracts/test-utils/src/log_utils.ts

@@ -2,8 +2,6 @@ import { LogEntry, LogWithDecodedArgs, TransactionReceiptWithDecodedLogs } from
 
 import { expect } from './chai_setup';
 
-// tslint:disable no-unnecessary-type-assertion
-
 /**
  * Filter logs by event name/type.
  */

contracts/test-utils/src/mocha_blockchain.ts

@@ -8,8 +8,6 @@ import * as process from 'process';
 
 import { provider, providerConfigs, txDefaults, web3Wrapper } from './web3_wrapper';
 
-// tslint:disable: no-namespace only-arrow-functions no-unbound-method max-classes-per-file
-
 export type ISuite = mocha.ISuite;
 export type ISuiteCallbackContext = mocha.ISuiteCallbackContext;
 export type SuiteCallback = (this: ISuiteCallbackContext) => void;

contracts/test-utils/src/order_factory.ts

@@ -29,7 +29,7 @@ export class OrderFactory {
             salt: generatePseudoRandomSalt(),
             ...this._defaultOrderParams,
             ...customOrderParams,
-        } as Order; // tslint:disable-line:no-object-literal-type-assertion
+        } as Order;
         const orderHashBuff = orderHashUtils.getOrderHashBuffer(order);
         const signature = signingUtils.signMessage(orderHashBuff, this._privateKey, signatureType);
         const signedOrder = {

contracts/test-utils/src/type_encoding_utils.ts

@@ -9,7 +9,6 @@ export const typeEncodingUtils = {
         const base = 10;
         const formattedValue = new BN(value.toString(base));
         const encodedValue = ethUtil.toBuffer(formattedValue);
-        // tslint:disable-next-line:custom-no-magic-numbers
         const paddedValue = ethUtil.setLengthLeft(encodedValue, constants.WORD_LENGTH);
         return paddedValue;
     },

contracts/test-utils/test/mocha_blockchain.ts

@@ -13,11 +13,8 @@ blockchainTests('mocha blockchain extensions', env => {
             expect(env.provider).to.exist('');
             expect(env.txDefaults).to.exist('');
             expect(env.web3Wrapper).to.exist('');
-            // HACK(dorothy-zbornak): tslint seems to get confused by these assertions.
-            // tslint:disable: no-unbound-method
             expect(typeof env.getChainIdAsync).to.eq('function');
             expect(typeof env.getAccountAddressesAsync).to.eq('function');
-            // tslint:enable: no-unbound-method
         });
 
         it('initializes the test environment', async () => {
@@ -96,7 +93,6 @@ blockchainTests('mocha blockchain extensions', env => {
 
 function createHookedObject(obj: any, handler: (name: string) => void, methods: string[]): any {
     const hookedMethods = _.map(methods, methodName => {
-        // tslint:disable: only-arrow-functions
         return function (this: any, ...args: any[]): any {
             handler(methodName);
             return obj[methodName].call(this, ...args);

contracts/test-utils/test/order_hash_test.ts

@@ -42,7 +42,6 @@ describe('Order hashing', () => {
             // It's common for developers using javascript to provide the amounts
             // as strings. Since we eventually toString() the BigNumber
             // before encoding we should result in the same orderHash in this scenario
-            // tslint:disable-next-line:no-unnecessary-type-assertion
             const orderHash = orderHashUtils.getOrderHashHex({
                 ...order,
                 makerAssetAmount: '0',

contracts/test-utils/test/subtests/mocha_blockchain_1.ts

@@ -1,7 +1,6 @@
 import { expect } from '../../src/chai_setup';
 import { blockchainTests, BlockchainTestsEnvironment } from '../../src/mocha_blockchain';
 
-// tslint:disable: no-default-export completed-docs
 export function append(env: BlockchainTestsEnvironment): void {
     blockchainTests('imported subtests', subtestsEnv => {
         it('shares the same environment object', () => {

contracts/test-utils/test/transaction_hash.ts

@@ -35,7 +35,6 @@ describe('0x transaction hashing', () => {
             // It's common for developers using javascript to provide the amounts
             // as strings. Since we eventually toString() the BigNumber
             // before encoding we should result in the same orderHash in this scenario
-            // tslint:disable-next-line:no-unnecessary-type-assertion
             const transactionHash = transactionHashUtils.getTransactionHashHex({
                 ...transaction,
                 salt: '0',

contracts/test-utils/tsconfig.lint.json

@@ -1,7 +0,0 @@
-{
-    // This file is a workaround that issue: https://github.com/palantir/tslint/issues/4148#issuecomment-419872702
-    "extends": "./tsconfig",
-    "compilerOptions": {
-        "composite": false
-    }
-}

contracts/utils/test/lib_address_array.ts

@@ -142,4 +142,3 @@ describe('LibAddressArray', () => {
         });
     });
 });
-// tslint:disable:max-file-line-count

contracts/utils/test/lib_bytes.ts

@@ -446,7 +446,6 @@ blockchainTests('LibBytes', env => {
     describe('memCopy', () => {
         // Create memory 0x000102...FF
         const memSize = 256;
-        // tslint:disable:no-shadowed-variable
         const memory = new Uint8Array(memSize).map((_, i) => i);
         const memHex = toHex(memory);
 
@@ -782,4 +781,3 @@ blockchainTests('LibBytes', env => {
         });
     });
 });
-// tslint:disable:max-file-line-count

contracts/utils/test/lib_math.ts

@@ -310,7 +310,6 @@ blockchainTests('LibMath', env => {
                 const numerator = new BigNumber(100);
                 const denominator = new BigNumber(102);
                 const target = new BigNumber(52);
-                // tslint:disable-next-line: boolean-naming
                 const actual = await libsContract.isRoundingErrorFloor(numerator, denominator, target).callAsync();
                 expect(actual).to.eq(true);
             });
@@ -319,7 +318,6 @@ blockchainTests('LibMath', env => {
                 const numerator = new BigNumber(100);
                 const denominator = new BigNumber(101);
                 const target = new BigNumber(92);
-                // tslint:disable-next-line: boolean-naming
                 const actual = await libsContract.isRoundingErrorFloor(numerator, denominator, target).callAsync();
                 expect(actual).to.eq(false);
             });
@@ -328,9 +326,7 @@ blockchainTests('LibMath', env => {
                 const numerator = ONE_ETHER;
                 const denominator = ONE_ETHER.dividedToIntegerBy(2);
                 const target = ONE_ETHER.times(0.01);
-                // tslint:disable-next-line: boolean-naming
                 const expected = isRoundingErrorFloor(numerator, denominator, target);
-                // tslint:disable-next-line: boolean-naming
                 const actual = await libsContract.isRoundingErrorFloor(numerator, denominator, target).callAsync();
                 expect(actual).to.eq(expected);
             });
@@ -376,7 +372,6 @@ blockchainTests('LibMath', env => {
                 const numerator = new BigNumber(100);
                 const denominator = new BigNumber(101);
                 const target = new BigNumber(92);
-                // tslint:disable-next-line: boolean-naming
                 const actual = await libsContract.isRoundingErrorCeil(numerator, denominator, target).callAsync();
                 expect(actual).to.eq(true);
             });
@@ -385,7 +380,6 @@ blockchainTests('LibMath', env => {
                 const numerator = new BigNumber(100);
                 const denominator = new BigNumber(102);
                 const target = new BigNumber(52);
-                // tslint:disable-next-line: boolean-naming
                 const actual = await libsContract.isRoundingErrorCeil(numerator, denominator, target).callAsync();
                 expect(actual).to.eq(false);
             });
@@ -394,9 +388,7 @@ blockchainTests('LibMath', env => {
                 const numerator = ONE_ETHER;
                 const denominator = ONE_ETHER.dividedToIntegerBy(2);
                 const target = ONE_ETHER.times(0.01);
-                // tslint:disable-next-line: boolean-naming
                 const expected = isRoundingErrorCeil(numerator, denominator, target);
-                // tslint:disable-next-line: boolean-naming
                 const actual = await libsContract.isRoundingErrorCeil(numerator, denominator, target).callAsync();
                 expect(actual).to.eq(expected);
             });

contracts/utils/test/log_decoding.ts

@@ -53,7 +53,6 @@ describe('TestLogDecoding', () => {
                 .emitEvent()
                 .awaitTransactionSuccessAsync();
             expect(txReceipt.logs.length).to.be.equal(1);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[0] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.deep.equal(expectedEvent);
         });
         it('should not decode event args when no dependencies are passed into wrapper', async () => {
@@ -61,7 +60,6 @@ describe('TestLogDecoding', () => {
                 .emitEventDownstream()
                 .awaitTransactionSuccessAsync();
             expect(txReceipt.logs.length).to.be.equal(1);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[0] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.undefined();
         });
         it('should decode args for local but not downstream event when no dependencies are passed into wrapper', async () => {
@@ -69,15 +67,12 @@ describe('TestLogDecoding', () => {
                 .emitEventsLocalAndDownstream()
                 .awaitTransactionSuccessAsync();
             expect(txReceipt.logs.length).to.be.equal(2);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[0] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.deep.equal(expectedEvent);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[1] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.undefined();
         });
         it('should decode locally emitted event args when dependencies are passed into wrapper', async () => {
             const txReceipt = await testLogDecodingWithDependencies.emitEvent().awaitTransactionSuccessAsync();
             expect(txReceipt.logs.length).to.be.equal(1);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[0] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.deep.equal(expectedEvent);
         });
         it('should decode downstream event args when dependencies are passed into wrapper', async () => {
@@ -85,7 +80,6 @@ describe('TestLogDecoding', () => {
                 .emitEventDownstream()
                 .awaitTransactionSuccessAsync();
             expect(txReceipt.logs.length).to.be.equal(1);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[0] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.deep.equal(
                 expectedDownstreamEvent,
             );
@@ -95,9 +89,7 @@ describe('TestLogDecoding', () => {
                 .emitEventsLocalAndDownstream()
                 .awaitTransactionSuccessAsync();
             expect(txReceipt.logs.length).to.be.equal(2);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[0] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.deep.equal(expectedEvent);
-            // tslint:disable no-unnecessary-type-assertion
             expect((txReceipt.logs[1] as LogWithDecodedArgs<DecodedLogArgs>).args).to.be.deep.equal(
                 expectedDownstreamEvent,
             );

contracts/zero-ex/src/bloom_filter_utils.ts

@@ -21,7 +21,6 @@ function bitwiseOrWords(a: string, b: string): string {
     const bBits = hexWordToBitArray(b);
     const resultBits = aBits.slice();
     for (let i = 0; i < 256; ++i) {
-        // tslint:disable-next-line: no-bitwise
         resultBits[i] |= bBits[i];
     }
     return bitArrayToHexWord(resultBits);

contracts/zero-ex/src/migration.ts

@@ -19,8 +19,6 @@ import {
     ZeroExContract,
 } from './wrappers';
 
-// tslint:disable: completed-docs
-
 /**
  * Addresses of minimum features for a deployment of the Exchange Proxy.
  */

contracts/zero-ex/src/nonce_utils.ts

@@ -20,7 +20,6 @@ export function rlpEncodeNonce(nonce: number): string {
         return ethjs.bufferToHex(ethjs.toBuffer(nonce));
     } else {
         const rlpNonce = ethjs.toBuffer(nonce);
-        // tslint:disable-next-line: restrict-plus-operands
         return hexUtils.concat(rlpNonce.length + 0x80, ethjs.bufferToHex(rlpNonce));
     }
 }

contracts/zero-ex/test/features/selector_collision_test.ts

@@ -3,8 +3,6 @@ import { MethodAbi } from 'ethereum-types';
 
 import * as wrappers from '../../src/wrappers';
 
-// tslint:disable:no-string-literal
-
 blockchainTests('Selector collision test', env => {
     it('Function selectors do not collide', () => {
         const selectorToSignature: { [selector: string]: string } = {};

contracts/zero-ex/test/transformers/fill_quote_transformer_test.ts

@@ -272,7 +272,6 @@ blockchainTests.resets('FillQuoteTransformer', env => {
             };
         }
 
-        // tslint:disable-next-line: prefer-for-of
         for (let i = 0; i < data.fillSequence.length; ++i) {
             const orderType = data.fillSequence[i];
             if (data.side === Side.Sell) {

contracts/zero-ex/tslint.json

@@ -1,13 +0,0 @@
-{
-    "extends": ["@0x/tslint-config"],
-    "rules": {
-        "custom-no-magic-numbers": false,
-        "max-file-line-count": false,
-        "no-non-null-assertion": false,
-        "no-unnecessary-type-assertion": false,
-        "number-literal-format": false
-    },
-    "linterOptions": {
-        "exclude": ["src/artifacts.ts", "test/artifacts.ts"]
-    }
-}

lerna.json

@@ -4,7 +4,7 @@
     "version": "independent",
     "command": {
         "publish": {
-            "ignoreChanges": ["test/**/*", "*.md", "scripts", "lib", "tslint.json", "tsconfig.json"]
+            "ignoreChanges": ["test/**/*", "*.md", "scripts", "lib", "tsconfig.json"]
         }
     },
     "npmClient": "yarn",

package.json

@@ -48,7 +48,7 @@
         "bundlewatch": "bundlewatch",
         "lint:ts": "wsrun --fast-exit --parallel --exclude-missing -p $PKG -c lint",
         "lint": "yarn lint:ts && yarn lint:contracts",
-        "upgrade_tools": "node node_modules/@0x/monorepo-scripts/lib/upgrade_deps.js -p '@0x/abi-gen|@0x/assert|@0x/base-contract|@0x/contracts-gen|@0x/dev-utils|@0x/json-schemas|@0x/monorepo-scripts|@0x/sol-compiler|@0x/sol-coverage|@0x/sol-doc|@0x/sol-profiler|@0x/sol-resolver|@0x/sol-trace|@0x/sol-tracing-utils|@0x/subproviders|@0x/tslint-config|@0x/types|@0x/typescript-typings|@0x/utils|@0x/verdaccio|@0x/web3-wrapper|ethereum-types'",
+        "upgrade_tools": "node node_modules/@0x/monorepo-scripts/lib/upgrade_deps.js -p '@0x/abi-gen|@0x/assert|@0x/base-contract|@0x/contracts-gen|@0x/dev-utils|@0x/json-schemas|@0x/monorepo-scripts|@0x/sol-compiler|@0x/sol-coverage|@0x/sol-doc|@0x/sol-profiler|@0x/sol-resolver|@0x/sol-trace|@0x/sol-tracing-utils|@0x/subproviders|@0x/types|@0x/typescript-typings|@0x/utils|@0x/verdaccio|@0x/web3-wrapper|ethereum-types'",
         "upgrade_deps": "node node_modules/@0x/monorepo-scripts/lib/upgrade_deps.js",
         "verdaccio": "docker run --rm -i -p 4873:4873 0xorg/verdaccio"
     },

packages/contract-addresses/tslint.json

@@ -1,3 +0,0 @@
-{
-    "extends": ["@0x/tslint-config"]
-}

packages/contract-artifacts/tslint.json

@@ -1,3 +0,0 @@
-{
-    "extends": ["@0x/tslint-config"]
-}

packages/contract-wrappers/src/generated-wrappers/broker.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class BrokerContract extends BaseContract {
     /**
      * @ignore
@@ -515,7 +508,7 @@ export class BrokerContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = BrokerContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = BrokerContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -942,7 +935,3 @@ export class BrokerContract extends BaseContract {
         });
     }
 }
-
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/coordinator.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class CoordinatorContract extends BaseContract {
     /**
      * @ignore
@@ -539,7 +532,7 @@ export class CoordinatorContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = CoordinatorContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = CoordinatorContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -1002,6 +995,3 @@ export class CoordinatorContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/dev_utils.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class DevUtilsContract extends BaseContract {
     /**
      * @ignore
@@ -1556,7 +1549,7 @@ export class DevUtilsContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = DevUtilsContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = DevUtilsContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -3434,6 +3427,3 @@ export class DevUtilsContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/erc20_token.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type ERC20TokenEventArgs = ERC20TokenApprovalEventArgs | ERC20TokenTransferEventArgs;
 
@@ -57,9 +53,6 @@ export interface ERC20TokenTransferEventArgs extends DecodedLogArgs {
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class ERC20TokenContract extends BaseContract {
     /**
      * @ignore
@@ -387,7 +380,7 @@ export class ERC20TokenContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = ERC20TokenContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = ERC20TokenContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -794,7 +787,3 @@ export class ERC20TokenContract extends BaseContract {
         });
     }
 }
-
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/erc721_token.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type ERC721TokenEventArgs =
     | ERC721TokenApprovalEventArgs
@@ -67,9 +63,6 @@ export interface ERC721TokenTransferEventArgs extends DecodedLogArgs {
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class ERC721TokenContract extends BaseContract {
     /**
      * @ignore
@@ -477,7 +470,7 @@ export class ERC721TokenContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = ERC721TokenContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = ERC721TokenContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -1078,6 +1071,3 @@ export class ERC721TokenContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/exchange.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type ExchangeEventArgs =
     | ExchangeAssetProxyRegisteredEventArgs
@@ -123,9 +119,6 @@ export interface ExchangeTransactionExecutionEventArgs extends DecodedLogArgs {
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class ExchangeContract extends BaseContract {
     /**
      * @ignore
@@ -3187,7 +3180,7 @@ export class ExchangeContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = ExchangeContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = ExchangeContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -6334,6 +6327,3 @@ export class ExchangeContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/forwarder.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type ForwarderEventArgs = ForwarderOwnershipTransferredEventArgs;
 
@@ -49,9 +45,6 @@ export interface ForwarderOwnershipTransferredEventArgs extends DecodedLogArgs {
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class ForwarderContract extends BaseContract {
     /**
      * @ignore
@@ -739,7 +732,7 @@ export class ForwarderContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = ForwarderContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = ForwarderContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -1631,6 +1624,3 @@ export class ForwarderContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/gods_unchained_validator.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class GodsUnchainedValidatorContract extends BaseContract {
     /**
      * @ignore
@@ -241,7 +234,7 @@ export class GodsUnchainedValidatorContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = GodsUnchainedValidatorContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = GodsUnchainedValidatorContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -330,7 +323,3 @@ export class GodsUnchainedValidatorContract extends BaseContract {
         });
     }
 }
-
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/i_asset_data.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class IAssetDataContract extends BaseContract {
     /**
      * @ignore
@@ -310,7 +303,7 @@ export class IAssetDataContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = IAssetDataContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = IAssetDataContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -790,6 +783,3 @@ export class IAssetDataContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/i_liquidity_provider.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class ILiquidityProviderContract extends BaseContract {
     /**
      * @ignore
@@ -291,7 +284,7 @@ export class ILiquidityProviderContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = ILiquidityProviderContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = ILiquidityProviderContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -506,6 +499,3 @@ export class ILiquidityProviderContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/i_transform_erc20.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type ITransformERC20EventArgs =
     | ITransformERC20QuoteSignerUpdatedEventArgs
@@ -65,9 +61,6 @@ export interface ITransformERC20TransformerDeployerUpdatedEventArgs extends Deco
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class ITransformERC20Contract extends BaseContract {
     /**
      * @ignore
@@ -455,7 +448,7 @@ export class ITransformERC20Contract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = ITransformERC20Contract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = ITransformERC20Contract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -1120,6 +1113,3 @@ export class ITransformERC20Contract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/i_zero_ex.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type IZeroExEventArgs =
     | IZeroExERC1155OrderCancelledEventArgs
@@ -266,9 +262,6 @@ export interface IZeroExTransformerDeployerUpdatedEventArgs extends DecodedLogAr
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class IZeroExContract extends BaseContract {
     /**
      * @ignore
@@ -7189,7 +7182,7 @@ export class IZeroExContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = IZeroExContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = IZeroExContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -15479,6 +15472,3 @@ export class IZeroExContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/maximum_gas_price.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -33,12 +30,8 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class MaximumGasPriceContract extends BaseContract {
     /**
      * @ignore
@@ -213,7 +206,7 @@ export class MaximumGasPriceContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = MaximumGasPriceContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = MaximumGasPriceContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -326,6 +319,3 @@ export class MaximumGasPriceContract extends BaseContract {
     }
 }
 
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/staking.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type StakingEventArgs =
     | StakingAuthorizedAddressAddedEventArgs
@@ -168,9 +164,6 @@ export interface StakingUnstakeEventArgs extends DecodedLogArgs {
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class StakingContract extends BaseContract {
     /**
      * @ignore
@@ -1656,7 +1649,7 @@ export class StakingContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = StakingContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = StakingContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -3810,7 +3803,3 @@ export class StakingContract extends BaseContract {
         });
     }
 }
-
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/staking_proxy.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type StakingProxyEventArgs =
     | StakingProxyAuthorizedAddressAddedEventArgs
@@ -74,9 +70,6 @@ export interface StakingProxyStakingContractAttachedToProxyEventArgs extends Dec
 export interface StakingProxyStakingContractDetachedFromProxyEventArgs extends DecodedLogArgs {}
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class StakingProxyContract extends BaseContract {
     /**
      * @ignore
@@ -783,7 +776,7 @@ export class StakingProxyContract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = StakingProxyContract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = StakingProxyContract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -1801,7 +1794,3 @@ export class StakingProxyContract extends BaseContract {
         });
     }
 }
-
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/src/generated-wrappers/weth9.ts

@@ -1,6 +1,3 @@
-// tslint:disable:no-consecutive-blank-lines ordered-imports align trailing-comma enum-naming
-// tslint:disable:whitespace no-unbound-method no-trailing-whitespace
-// tslint:disable:no-unused-variable
 import {
     AwaitTransactionSuccessOpts,
     EncoderOverrides,
@@ -35,7 +32,6 @@ import { EventCallback, IndexedFilterValues, SimpleContractArtifact } from '@0x/
 import { Web3Wrapper } from '@0x/web3-wrapper';
 import { assert } from '@0x/assert';
 import * as ethers from 'ethers';
-// tslint:enable:no-unused-variable
 
 export type WETH9EventArgs =
     | WETH9ApprovalEventArgs
@@ -73,9 +69,6 @@ export interface WETH9WithdrawalEventArgs extends DecodedLogArgs {
 }
 
 /* istanbul ignore next */
-// tslint:disable:array-type
-// tslint:disable:no-parameter-reassignment
-// tslint:disable-next-line:class-name
 export class WETH9Contract extends BaseContract {
     /**
      * @ignore
@@ -511,7 +504,7 @@ export class WETH9Contract extends BaseContract {
 
     public getFunctionSignature(methodName: string): string {
         const index = this._methodABIIndex[methodName];
-        const methodAbi = WETH9Contract.ABI()[index] as MethodAbi; // tslint:disable-line:no-unnecessary-type-assertion
+        const methodAbi = WETH9Contract.ABI()[index] as MethodAbi;
         const functionSignature = methodAbiToFunctionSignature(methodAbi);
         return functionSignature;
     }
@@ -1068,7 +1061,3 @@ export class WETH9Contract extends BaseContract {
         });
     }
 }
-
-// tslint:disable:max-file-line-count
-// tslint:enable:no-unbound-method no-parameter-reassignment no-consecutive-blank-lines ordered-imports align
-// tslint:enable:trailing-comma whitespace no-trailing-whitespace

packages/contract-wrappers/tslint.json

@@ -1,3 +0,0 @@
-{
-    "extends": ["@0x/tslint-config"]
-}