Code review comment for lp:~gmb/launchpad/reschedule-button-qafix-bug-558415

Revision history for this message
Brad Crittenden (bac) wrote :

Hi Graham this branch looks good. You have one typo of 'wathc'.

Also I know you just copied it from the existing code, but I'd expect to see

return failure_ratio < WATCH_RESCHEDULE_THRESHOLD

So that 6/10 failures would still return true.

review: Approve (code)

« Back to merge proposal