Closed keegancsmith closed 8 years ago
LGTM -- but CI looks borked due to something unrelated(?)
@slimsag I can't actually view the CI for this for some reason. But I would assume it is unrelated. I ran all the tests locally. If you think it is unrelated, then I'll merge. Is that cool?
Yeah -- no clue, it's gotta be unrelated
let's ship!
Also updated the example test so it passes.
This PR originally tried to improve on our error handling, but I'm leaving that out for now and just including the related fixes.