Re: [v2 PATCH 2/2] reset: add tests for git reset -

2015-03-10 Thread Eric Sunshine
On Tue, Mar 10, 2015 at 11:38 AM, Sundararajan R wrote: > reset: add tests for git reset - Since this patch is changing the tests rather than 'reset' itself, you'd likely want to say: t7102: add 'reset -' tests > The failure case which occurs on teaching git is taught the '-' shorthand > is

Re: [v2 PATCH 2/2] reset: add tests for git reset -

2015-03-10 Thread Torsten Bögershausen
On 2015-03-10 16.38, Sundararajan R wrote: > Helped-by: Torsten Bögershausen There seems to be an issue that the mail is encoded from (what ? Latin-1) into UTF-8 2 times The easy solution is to remove the line, I'm OK with that, since a review-comment is not necessarily motivating a Helped-by,