[darcs-devel] [issue2042] repair should be a subcommand of check

[gh]

New submission from gh <guillaumh at gmail.com>:

I suggest to make the repair command a subcommand of check, that should
be invoked with "darcs check --repair".

Conceptually, repairing something is a proper supertask of checking
whether it is broken. Otherwise, how could you know what to repair?

When it comes to the current darcs code, modules Darcs.Command.Check and
Darcs.Command.Repair have a lot of overlap, and Darcs.Commands.Repair is
quite small (without the boilerplate).

Making repair an option of check would only involve making sure tests
are not run (ie, --repair should imply --no-test).

This change would also get rid of the module Darcs.Command.Repair, for
the greatest cognitive relief of current and future darcs developers. :-)

And "darcs repair" should be added as an alias for "darcs check --repair".

----------
messages: 13662
nosy: gh, jaredj
priority: wishlist
status: unknown
title: repair should be a subcommand of check
topic: ProbablyEasy, UI

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/issue2042>
__________________________________