Testing patches at record time is nice, but it can really disrupt the workflow,
and the temptation is always great to use --no-test. But using --no-test is not
satisfactory either: who as never sent or pushed a junk patch that way. I
propose that a line 'Untested' be added to the description of any patch recorded
with --no-test. It can then be removed by running amend-record --edit and testing.
When sending and pushing, darcs should test any patch that has the Untested tag
(one by one). pull should also warn about pulling untested patches.
|