Thread (1 message) 1 message, 1 author, 2018-06-27

Re: [PATCH v2 1/2] t3418: add testcase showing problems with rebase -i and strategy options

From: Junio C Hamano <hidden>
Date: 2018-06-27 19:24:05

Elijah Newren [off-list ref] writes:
Seems reasonable.  Since these tests were essentially copies of other
tests within the same file, just for rebase -i instead of -m, should I
also add another patch to the series fixing up the rebase -m testcase
to also replace the subshell with a single-shot environment
assignment?
I personally would think it would be best to leave that to a later
day, and take your v3 that properly &&-chains these two assignments.

It may be a good clean-up, but is an overkill if done as a
preparatory clean-up in the context of these two small fixes.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help