Junio C Hamano <gitster@xxxxxxxxx> writes:
>> Looks good :) Just a note that this is in
>> "ab/only-single-progress-at-once" already in case you didn't spot it.
>>
>> That series is marked for "next?" (with a question mark) by Junio in the
>> latest what's cooking, so *hint* *hint* for any last minute review :)
>
> I wonder if it would help contributors if we give them a simple rule
> to follow before sending their patches out:
>
> - You develop on an appropriate base (either on maint, or master,
> or a merge of selected topics in flight you absolutely have to
> depend on) as usual.
>
> - You test merge the result to 'seen' and to 'next'. If you do not
> get heavy conflicts and the tests pass, you are OK.
>
> - Otherwise, you study what is conflicting, and should review it
> before you submit your topic.
Addendum to this third step.
... Also, if you chose to depend on other topics when you
decided on the base in the first step, giving them your review
would be a good way to show others that _you_ understand the
code that you are building on top of, which in turn may give
others confidence in your topic.
Thanks.
