login
Header Space

 
 

Re: [PATCH 5/5] Introduce fast forward option only

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Sverre Hvammen Johansen <hvammen@...>
Cc: Junio C Hamano <gitster@...>, <git@...>
Date: Friday, April 25, 2008 - 6:46 am

"Sverre Hvammen Johansen" <hvammen@gmail.com> writes:


I'd rather have description of the change in the commit message
itself.  You can add "See the documentation for details", but the
basic description, like:

   This commit introcudes fast forward option 'only'.  With --ff=only
   merge succeeds only if it resolves to fast-forward merge.
 
It would be nice to have described either in commit message, or in the
documentation, _why_ it is needed for integration with AccuRev,
i.e. what features of AccuRev or of integration requires this.

-- 
Jakub Narebski
Poland
ShadeHawk on #git
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 5/5] Introduce fast forward option only, Sverre Hvammen Johansen, (Thu Apr 24, 1:53 am)
Re: [PATCH 5/5] Introduce fast forward option only, Jakub Narebski, (Fri Apr 25, 6:46 am)
Re: [PATCH 5/5] Introduce fast forward option only, Junio C Hamano, (Fri Apr 25, 2:59 pm)
Re: [PATCH 5/5] Introduce fast forward option only, Sverre Hvammen Johansen, (Fri Apr 25, 5:00 pm)
speck-geostationary