Retry() does not respect context cancellation

[CodyDWJones]

A context cancellation / missed deadline is permanent, and if it happens while retrying, there is no point in making further attempts. However, Retry() continues trying even if the developer's action returns a non-nil error from Context.Err(). It also sleeps for the full duration instead of waking up immediately.

[kamilsk]

Hi! For that reason I made a fork: https://github.com/kamilsk/retry

[CodyDWJones]

I didn't know about your fork and will check that out - thank you!