Thread (97 messages) 97 messages, 6 authors, 2021-02-06

Re: [PATCH v5 0/8][Outreachy] rebase -i: add options to fixup command

flat view

From: Christian Couder <hidden>
Date: 2021-02-05 18:29:32

On Fri, Feb 5, 2021 at 10:42 AM Charvi Mendiratta [off-list ref] wrote:
On Fri, 5 Feb 2021 at 13:00, Eric Sunshine [off-list ref] wrote:
quoted
quoted
* changed the flag type from enum todo_item_flags to unsigned
* Replaced fixup_-* with fixup-* in lib-rebase.sh
* fixup a small nit in Documentation
These changes are still worthwhile and can easily be done
incrementally atop what is already in next, I would think.
I agree, these fixes are required. So, I am not sure but now to do these
fixup shall I send another patch cleaning this patch series(v4) and rebase the
patch on the 'next' branch ? Is it the right way ?
Yeah, I think you can send each of the above 3 changes in a different
patch on top of the 'next' branch. That would create a new 3 patch
long series, which you should give a new name and not consider v5 of
the previous patch series.

Best,
Christian.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help