On 07/27/2015 04:14 AM, vexed wrote:
> On Sun, Jul 26, 2015 at 2:32 PM, Kreuvf  wrote:
>> On 07/26/2015 04:27 AM, vexed wrote:
>>>
>>>   Log Message:
>>>   -----------
>>>   Hideous indeed.
>>> Bad things happen when player < 6.
>>
>> Is this a case of https://xkcd.com/1296/? Please be more descriptive in your
>> commit messages, there are people who actually read them :)
>>
>> Best regards
>> - Kreuvf
> 
> That *was* being descriptive though... if player's value was less than
> 6, then, we spam errors.
> You could also look at the commit in question, and see that all that
> was removed was, "- 6", then think, yeah, I understand.

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".

Best regards
- Kreuvf


Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
_______________________________________________
Warzone2100-project mailing list
Warzone2100-project@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/warzone2100-project

Reply via email to