Antonio Ospite <ospite@xxxxxxxxxxxxxxxxx> writes:
> ... About the tests, I am going to modify one of your
> tests instead of adding another one, is that OK?
It is not just Ok but is preferred; that is a good way to document where
the behaviour changed how, and for what reason ;-).
Perhaps an illustration in the documentation may help. Until I read what
Matthieu wrote in his message, I didn't quite get why anybody wanted this
new behaviour---my understanding of which is to get something like this:
[PATCH 0/2] Here is what I did...
[PATCH 1/2] Clean up and tests
[PATCH 2/2] Implementation
[PATCH v2 0/3] Here is a reroll
[PATCH v2 1/3] Clean up
[PATCH v2 2/3] New tests
[PATCH v2 3/3] Implementation
when sending the re-rolled series, with the --in-reply-to for [v2 0/3] set
to [0/2] of the original. If you illustrate the current behaviour in a
similar way in your commit log message, perhaps side-by-side to save
vertical space, it would help make it clear why people would want the new
behaviour.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html