> Style and Haddock for Darcs.Commands.Test
Small style tweaks.
> Use safeApply throughout Darcs.Commands.Test
Good idea.
> Unify darcs test strategy type signatures
Useful for next patch (could have been merged?).
> Factor out Strategy type
OK.
> Clearer test strategy selection
Good.
> Consistent test strategy function naming
Naming cleanup.
> Correct typos in trackdown-bisect test comments
OK.
> Improve darcs test descriptive help
OK.
> Factor out initial test run
Nice one, every test strategy does the test on the current state of the
repo, and then chooses proceeds (or not, as it is the case for --once).
> Factor out function to build bisect strategy initial state
Why not.
> Implement exponential backoff test strategy
The new strategy is well commented, thanks for that.
Tests pass, so I'm accepting the bundle. Thanks!
This new strategy needs a high-level explanation on the wiki (like the
one for bisect).
|