Pass container on success in #run

[lightsofapollo]

No description provided.

[lightsofapollo]

Fixes #20 also fixes some bugs where error callbacks don't return early.

[apocas]

Regarding the remove, at the time I left it async mainly for two reasons:
First is that removal isn't really that important for the run objective (think of it like garbage collection done in the 'background')
Second is that container removal in really slow in some scenarios.

But as usual I'm open to opinions, we can't make it another optional specification, there's already many options in run.

What do you think?

[lightsofapollo]

I think those are actually good arguments for deleting the temp flag and allowing the caller to deal with removing the container when it makes sense to do so. See #22.

I would consider this PR generally a bugfix so ideally we could land this and discuss removal there.