Address feedback

src/ts/0x.js.ts

@@ -132,8 +132,9 @@ export class ZeroEx {
 
     /**
      * Converts BigNumber instance to BN
-     * We do it because ethABI accepts only BN's
-     * We should be consistent about using BigNumbers in our codebase and not use BN anywhere else
+     * The only we convert to BN is to remain compatible with `ethABI. soliditySHA3 ` that
+     * expects values of Solidity type `uint` to be of type `BN`.
+     * We do not use BN anywhere else in the codebase.
      */
     private static bigNumberToBN(value: BigNumber.BigNumber) {
         return new BN(value.toString(), 10);

src/ts/globals.d.ts

@@ -26,5 +26,5 @@ declare module 'ethereumjs-util' {
 }
 
 declare module 'ethereumjs-abi' {
-    const soliditySHA3: (argTypes: string[], args: any) => Buffer;
+    const soliditySHA3: (argTypes: string[], args: any[]) => Buffer;
 }

src/ts/types.ts

@@ -13,8 +13,5 @@ function strEnum(values: string[]): {[key: string]: string} {
 export const SolidityTypes = strEnum([
     'address',
     'uint256',
-    'uint8',
-    'string',
-    'bool',
 ]);
 export type SolidityTypes = keyof typeof SolidityTypes;