Closed armcknight closed 10 years ago
@ndonald2 Hit all points.
Looks good to me. @ZevEisenberg ?
@ndonald2 @ZevEisenberg fixed documentation and failure cases
@ZevEisenberg fixed
@ZevEisenberg Ah, good catch, got it.
@ZevEisenberg done.
Much cleaner! :thumbsup:
@ndonald2 This looks good to merge, if you'd like to do the honors.
@ndonald2 This feature was part of a large dangling branch, but we need it in for our project, so I created a new branch just for this. There may be one or two more of these coming your way.