Inconsistent default retries about node-attempt HOT 3 CLOSED

Thanks for pointing this out! Waiting for this resolution before I can publish the fix: npm/npm#7520

from node-attempt.

Diverging a bit, but isn't 2 retries meaning it may actually do the operation 3 times? In that case I'd suggest to change so that it's the total number of attempts (i.e., 3 or 6 by default).

from node-attempt.

NPM and Github are now in sync with default retries. Thanks for the wording suggestion! I've updated the README on Github with the clarification (will go to NPM with the next patch). I'm open to deprecating the 'retries' option and, instead, sending in something like totalAttempts, but since that would be a minor release instead of just a patch, I wanted to get the patch out now.

from node-attempt.