💨 Add partial jitter #14
Conversation
Sometimes you want jitter but only a bit of it: closer to the non-jittered value than you'd have got with full jitter. Backwards compatible with current "full jitter" mode. Also, regenerate package lock.
| @@ -81,7 +90,7 @@ export async function retry( | |||
| options: BackoffOptions, | |||
| targetFunction: any, | |||
| thisArg: any = null, | |||
| ...args: any[], | |||
| ...args: any[] // tslint:disable-line | |||
There was a problem hiding this comment.
tsc's A rest parameter or binding pattern may not have a trailing comma.ts(1013) was fighting with tslint's trailing-comma rule
| @@ -81,7 +90,7 @@ export async function retry( | |||
| options: BackoffOptions, | |||
| targetFunction: any, | |||
| thisArg: any = null, | |||
| ...args: any[], | |||
| ...args: any[] // tslint:disable-line | |||
There was a problem hiding this comment.
Look like a bug in tslint: palantir/tslint#4172
According to that issue, another workaround is to add esSpecCompliant: true to the trailing-comma rule.
There was a problem hiding this comment.
Look like a bug in tslint: palantir/tslint#4172
According to that issue, another workaround is to addesSpecCompliant: trueto thetrailing-commarule.
@erykwarren hmmm I can't get the option to work, it keeps complaining (and it's supposed to be true by default). Not going to dig more, leaving the disable-line, anyway tslint is getting replaced by eslint.
Sometimes you want jitter but only a bit of it: closer to the
non-jittered value than you'd have got with full jitter.
Backwards compatible with current "full jitter" mode.
Also, regenerate package lock.