[webkit-dev] Proposal: Stop EWS bot commenting in bugs

[Alexey Proskuryakov]

(re-sent from a correct address)

I think that it's good to try not dumping build failures into comments right away, and to see what happens.

As for not showing style bot failures, it seems almost certain that this will make them substantially more annoying to work with. Can you describe the workflow for patch author and reviewer to deal with style bot warnings when they are not inline? Manually finding relevant lines by number can't work.

I agree with Tim that dumping all tested paths along with style warnings is silly. How hard would it be it to get rid of that?

- WBR, Alexey Proskuryakov

15 янв. 2014 г., в 20:34, Ryosuke Niwa <rniwa at webkit.org> написал(а):

> FWIW, I've posted a patch to implement the compromise Sam proposed: https://bugs.webkit.org/show_bug.cgi?id=127079
> 
> It would be nice to update the code-view.js so that it could show style errors inline next to code but that could be done in a separate patch.
> 
> - R. Niwa
> 
> 
> On Wed, Jan 15, 2014 at 8:29 PM, Gyuyoung Kim <gyuyoung.kim at webkit.org> wrote:
> +1, I would prefer to see EWS log only when I want to see it.
> 
> Gyuyoung.
> 
> 
> On Thu, Jan 16, 2014 at 1:24 PM, Joseph Pecoraro <pecoraro at apple.com> wrote:
> I would also like to see a reduction in EWS "spam".
> 
> It is not just the comment clutter, but also quite a bit of emails.
> 
> - Joe
> 
> On Jan 15, 2014, at 8:17 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
> 
>> We could do that, or add some JS hack to Bugzilla so that it hides EWS comments by default but makes them expandable.
>> 
>> - R. Niwa
>> 
>> 
>> On Wed, Jan 15, 2014 at 8:09 PM, Sam Weinig <weinig at apple.com> wrote:
>> Could we compromise for now, and remove all the non-test failing EWS comments (e.g. build failure, style failure)?
>> 
>> - Sam
>> 
>> On Jan 15, 2014, at 8:04 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
>> 
>>> I find Mac EWS's listing the failing tests to be very useful especially because it uploads the results to Bugzilla.
>>> 
>>> I do agree that comments about build failures are much less useful.
>>> 
>>> - R. Niwa
>>> 
>>> 
>>> On Wed, Jan 15, 2014 at 7:54 PM, Sam Weinig <weinig at apple.com> wrote:
>>> Hi Everyone,
>>> 
>>> I am becoming increasingly annoyed by the comments made in bugs.webkit.org bugs by our non-human helpers, the EWS bots.  I don’t find the addition of a comment indicating that a patch has failed on a bot, over the existing indication in the bubble, to be worth the noise it creates.
>>> 
>>> I propose that we stop allowing the bots to comment, and leave that space for the developers.
>>> 
>>> - Sam
>>> 
>>> _______________________________________________
>>> webkit-dev mailing list
>>> webkit-dev at lists.webkit.org
>>> https://lists.webkit.org/mailman/listinfo/webkit-dev
>>> 
>> 
>> 
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev at lists.webkit.org
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
> 
> 
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev
> 
> 
> 
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-dev/attachments/20140115/7abc1bf5/attachment.html>