Improve comments and remove unused imports

packages/contracts/test/exchange/transactions.ts

@@ -1,6 +1,6 @@
 import { BlockchainLifecycle } from '@0xproject/dev-utils';
 import { assetProxyUtils, generatePseudoRandomSalt } from '@0xproject/order-utils';
-import { AssetProxyId, Order, OrderWithoutExchangeAddress, SignatureType, SignedOrder } from '@0xproject/types';
+import { AssetProxyId, Order, OrderWithoutExchangeAddress, SignedOrder } from '@0xproject/types';
 import { BigNumber } from '@0xproject/utils';
 import * as chai from 'chai';
 import * as ethUtil from 'ethereumjs-util';

packages/order-utils/src/order_hash.ts

@@ -1,5 +1,5 @@
 import { schemas, SchemaValidator } from '@0xproject/json-schemas';
-import { Order, SignatureType, SignedOrder } from '@0xproject/types';
+import { Order, SignedOrder } from '@0xproject/types';
 import { BigNumber } from '@0xproject/utils';
 import BN = require('bn.js');
 import { SolidityTypes } from 'ethereum-types';

packages/order-utils/src/signature_utils.ts

@@ -16,8 +16,8 @@ import { utils } from './utils';
 /**
  * Verifies that the provided signature is valid according to the 0x Protocol smart contracts
  * @param   data          The hex encoded data signed by the supplied signature.
- * @param   signature     A hex encoded 0x Protocol signature made up of: [SignatureType][TypeSpecificData].
- *          E.g [SignatureType.EIP712][vrs]
+ * @param   signature     A hex encoded 0x Protocol signature made up of: [TypeSpecificData][SignatureType].
+ *          E.g [vrs][SignatureType.EIP712]
  * @param   signerAddress The hex encoded address that signed the data, producing the supplied signature.
  * @return  Whether the signature is valid for the supplied signerAddress and data.
  */

packages/types/src/index.ts

@@ -1,6 +1,9 @@
 import { BigNumber } from 'bignumber.js';
 import { ContractAbi, DecodedLogArgs, LogEntry, LogWithDecodedArgs, TransactionReceipt } from 'ethereum-types';
 
+// HACK: Rather then extending from OrderWithoutExchangeAddress
+// we don't, because our docs don't expand inherited types, and it's unnecessarily
+// confusing to introduce the user to the OrderWithoutExchangeAddress type.
 export interface Order {
     senderAddress: string;
     makerAddress: string;