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....@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 <pecor...@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 <rn...@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 <wei...@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 <rn...@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 <wei...@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@lists.webkit.org
>>>> https://lists.webkit.org/mailman/listinfo/webkit-dev
>>>>
>>>
>>>
>>>
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev@lists.webkit.org
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
>>
>>
>>
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev@lists.webkit.org
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
>>
>>
>
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to