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

[Timothy Hatcher]

It isn't just about seeing the comment on the page, it also gets emailed.

I agree, no bot comments would be better. The stye bot spew is particularly bad, mainly because it dumps all the file paths it tested not just what fails.

— Timothy Hatcher

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

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