+1

Sent from Mobile.

On Aug 14, 2013, at 9:25, Chris Peterson <cpeter...@mozilla.com> wrote:

> We could also send a weekly congratulations to the person who removed the 
> most lines of code that week. :)
>
> chris
>
>
> On 8/13/13 1:57 PM, Jet Villegas wrote:
>> This is awesome! Is it possible to see a log of the recipients/patches?
>>
>> --Jet
>>
>> ----- Original Message -----
>> From: "Matt Brubeck" <mbrub...@mozilla.com>
>> To: dev-platform@lists.mozilla.org
>> Sent: Monday, August 12, 2013 3:14:20 PM
>> Subject: Proposal: Email individual patch authors who improve performance
>>
>> I've posted a patch that would change how the graph server sends email
>> when a performance *improvement* is detected:
>> https://bugzilla.mozilla.org/show_bug.cgi?id=904250
>>
>> Emails about regressions would be unchanged.  Emails about improvements
>> would now be sent to individual patch authors using the same logic as
>> regression emails:
>>
>> * If there are up to five authors in the regression range, email
>> dev-tree-management and each of the patch authors.
>> * If there are more than five authors, just email dev-tree-management.
>>
>> I've spoken to other developers in a few places, and most were in favor
>> of this change.  Those who don't want the extra emails can filter them
>> out if they want to.  Motivations for this change:
>>
>> - When you are expecting a patch to affect performance, the email is
>> useful confirmation.  Receiving it directly saves you the work of
>> searching through the mailing list archives or manually inspecting graphs.
>>
>> - When you're not expecting a performance improvement, it may be
>> important to know that one has happened so you can figure out why.
>> (Sometimes it's a sign of a bug, for example if you accidentally
>> disabled some code.)
>>
>> - Warm fuzzy feeling for developers who write perf wins!
>>
>> I think the accuracy of the analysis script is now good enough that this
>> would not generate too many spurious emails.  If you have any opinions
>> or suggestions about this change, please share them here or in bug 904250!
>> _______________________________________________
>> dev-platform mailing list
>> dev-platform@lists.mozilla.org
>> https://lists.mozilla.org/listinfo/dev-platform
>
> _______________________________________________
> dev-platform mailing list
> dev-platform@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-platform
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to