I really like this pkg so far. Some feedback/ideas, as I'm testing it out: - [ ] hadle panic & report it as `last_error` (rethrow panic or provide onPanic hook?) - [ ] on error, I see `"last_error": null` in stats - [ ] Add runnable.Must(), which would bring down the whole group on error (e.g. HTTP server) - [ ] improve docs around `runnable.Retry()` -- it's not very clear what the resetAfter arg is for - [ ] add `uptime time.Duration` to Status - [ ] `run.New()` return *struct instead of an interface - [ ] `run.Group()` return *struct instead of an interface
I really like this pkg so far.
Some feedback/ideas, as I'm testing it out:
last_error(rethrow panic or provide onPanic hook?)"last_error": nullin statsrunnable.Retry()-- it's not very clear what the resetAfter arg is foruptime time.Durationto Statusrun.New()return *struct instead of an interfacerun.Group()return *struct instead of an interface