Closed mpchadwick closed 8 years ago
LGTM (sorry I merged out of order, now you have to rebase)
Actually I'm not sure there was a correct order that would allow both to be merged because they both modified the constructor of the abstract and both branched off master. I'll rebase this in a bit.
@philwinkle rebased.
@philwinkle hold up. The config model seems to have disappeared...
@philwinkle OK I added the config model. Should be all set.
LGTM!
Partial implementation off issue described in #23.