On Tue, Jul 28, 2015 at 12:13:37AM +0200, Kreuvf wrote: > > Yes, I _could_, but I rather look at commits, because the commit message > sparked > my interest, not because it puzzled me. > > > But, you mean, you rather me type something like... > > "We don't want negative players, WZ has enough of them as it is."? ;) > > > > Making it say "Fix issue where player value could be negative" is > > basically the same thing. > > I prefer "Fix issue where player value could be negative".
I'm with Kreuf here. The summary line should describe what the commit does. Clever stuff can go in the commit msg body. Having to look at the code to decipher the meaning is Fail. S. -- Stephen Swaney sswa...@centurytel.net 231-313-2492 skype: sswaney (GMT-4) ------------------------------------------------------------------------------ _______________________________________________ Warzone2100-project mailing list Warzone2100-project@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/warzone2100-project
