Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] t/t91XX-svn: start removing use of "git-" from these tests

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:45:20

Abhijit Menon-Sen [off-list ref] writes:
At 2008-09-08 03:27:59 -0700, normalperson@yhbt.net wrote:
quoted
They'll show up as false positives when grepping for "git-"
Fine, you're the maintainer, but for the record, I think this change is
dreadful. Changing all "git-svn blah" commands to "git svn blah" is one
thing, but obfuscating tests on the off-chance that someone might grep
for "git-" seems entirely wrong.
I think the value of this first one is not "on the off-chance" thing, but
the presense of this change makes it far easier to eyeball the result of
applying the second, large-ish patch.

If I were doing this as a series, I would probably have followed them up
with a patch that reverts the token replacement this first one does as the
final patch in the series, but I also think leaving this first patch as-is
is fine.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help