Hrm why did you close/delete this?

Was an accident. Too many commits were on the PR and wanted to clean them up.

You can just force push to clean them up.

Will do 👍

Usually what we've been doing is keeping open a [WIP] PR and would just push to it so people could track what we're doing. When we're ready to actually merge it back in we'd clean up the commits, force push it to the branch and then remove the [WIP] tag.

We're not always diligent about removing the tag though.

Should we re-open this then?

Apologize for the mess -- it's set up now as originally intended. On to solving the problem!

This should be rebased on top of #374 after it's merged. It's going to need the code from that for the callbacks to work properly.

Sounds like a plan 👍

#374 is merged. I rebased untracked on top of master.

The tests on Appveyor that are failing are the same ones that we're trying to fix, and are unrelated to this PR. I say we

Currently the tests fail in AppVeyor on master as well. Fixing those are out of scope of this PR and will be handled in a different one ASAP. That being said I don't think that's a valid reason for halting this right now since it's most likely a problem with the tests themselves and not the library.

Going to go ahead and merge this.