misc todo cleanup

Author: jeremydaly

README.md

@@ -385,7 +385,8 @@ attributes: {
 ```
 
 #### Using an `array` for composite keys
-- [ ] This needs some work
+
+**NOTE:** The interface for composite keys may be changing in v0.2 to make it easier to customize.
 
 Composite keys in DynamoDB are incredibly useful for creating hierarchies, one-to-many relationships, and other powerful querying capabilities (see [here](https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/bp-sort-keys.html)). The DynamoDB Toolbox lets you easily work with composite keys in a number of ways. In many cases, there is no need to store the data in the same record twice if you are already combining it into a single attribute. By using composite key mappings, you can store data together in a single field, but still be able to structure input data *and* parse the output into separate attributes.
 

__tests__/expressionBuilder.unit.test.js

@@ -1,5 +1,3 @@
-// TODO: Additional expression builder tests
-
 const expressionBuilder = require('../lib/expressionBuilder')
 
 // Require Table and Entity classes

classes/Entity.js

@@ -1,7 +1,5 @@
 'use strict'
 
-// TODO: make sure pks and sks are of correct types
-
 /**
  * DynamoDB Toolbox: A simple set of tools for working with Amazon DynamoDB
  * @author Jeremy Daly <[email protected]>
@@ -659,8 +657,6 @@ class Entity {
       returnValues ? { ReturnValues: returnValues.toUpperCase() } : null,
     ) // end assign
 
-    // console.log(payload)
-    
     return payload
 
     // TODO: Check why primary/secondary GSIs are using if_not_exists

index.js

@@ -7,11 +7,8 @@
  * @license MIT
  */
 
-
 // TODO: Check param merging
 // TODO: 'Remove null fields' option
-// TODO: Tests, tests, and more tests
-// TODO: Documentation updates 😳
 // TODO: prevent reserved field names?
 
 const Table = require('./classes/Table')

lib/expressionBuilder.js

@@ -95,7 +95,6 @@ const parseClause = (_clause,grp,table) => {
     attr, // attribute
     size, // wraps attr in size()
     negate, // negates expression
-    // TODO: thing about the name of this
     or, // change logical evaluator to OR
     eq, // =
     ne, // <>