Thread (4 messages) 4 messages, 2 authors, 2023-02-22

Re: [GSOC][PATCH] t4121: modernize test style

From: Vivan Garg <hidden>
Date: 2023-02-22 05:19:30

Possibly related (same subject, not in this thread)

On Tue, Feb 21, 2023 at 1:21 PM Junio C Hamano [off-list ref] wrote:
I am not Victoria, but I think

 * You had two (or more) changes in your patch.

 * She found one of them is good.

 * The other one(s) were found lacking.

When a patch is reviewed and found to be lacking (as a whole), and
if you agree with the issues pointed out by the review(s), you'd
redo the patch (with either "commit --amend" for a single patch, or
"rebase -i" for a series with multiple patches), and produce v2, as
if v1 never happened.  The act of doing so is colloquially called
"to reroll".

Documentation/MyFirstContribution.txt::[[now-what]] may have more
details.

Thanks.
That makes sense. Thanks for the clarification!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help