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

2014-01-28 Thread Eric Carlson

On Jan 27, 2014, at 5:39 PM, Chris Fleizach cfleiz...@apple.com wrote:

 I dislike this change now that's been rolled out. 
 
 The lack of email notices before confirmed that my patch was OK and I was 
 able to do something else while waiting for review.
 Now I have to continually revisit the bug page checking to see if more bots 
 have completed and that my patch is good.
 
 I think at least the person who submitted the patch should be notified when 
 there's been an error.
 
  I agree that it would be nice for the patch author to be notified when there 
is an error.

eric


___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2014-01-27 Thread Chris Fleizach
I dislike this change now that's been rolled out. 

The lack of email notices before confirmed that my patch was OK and I was able 
to do something else while waiting for review.
Now I have to continually revisit the bug page checking to see if more bots 
have completed and that my patch is good.

I think at least the person who submitted the patch should be notified when 
there's been an error.



On Jan 17, 2014, at 4:27 PM, Alexey Proskuryakov a...@webkit.org wrote:

 
 This has been implemented, and one unintended consequence is that this 
 noticeably affects how quickly one can iterate on time sensitive patches.
 
 It's a huge waste of time that you are no longer informed when a build fails 
 on EWS. This seriously delays urgent work, as you only start working on fixes 
 when you happen to manually poll, or even worse, when a reviewer tells you 
 about build breakage.
 
 It's good to not spam everyone, however patch author should be notified by 
 EWS immediately I think. Some ideas:
 
 - e-mail;
 - IRC;
 - browser notifications when bug page is open.
 
 The latter might be best, as it also gives some control over whether to get 
 pinged - keep the bug open if you care, close it if it's not urgent, and you 
 cannot afford distraction now.
 
 I filed https://bugs.webkit.org/show_bug.cgi?id=127203 about this.
 
 - WBR, Alexey Proskuryakov
 
 
 16 янв. 2014 г., в 15:09, Ryosuke Niwa rn...@webkit.org написал(а):
 
 Okay, let's remove the python paths but keep the style error messages until 
 we can improve the EWS infrastructure.
 
 - R. Niwa
 
 
 On Thu, Jan 16, 2014 at 9:41 AM, Timothy Hatcher timo...@apple.com wrote:
 On Jan 16, 2014, at 2:28 AM, Alexey Proskuryakov a...@webkit.org wrote:
 
 
 15 янв. 2014 г., в 23:02, Ryosuke Niwa rn...@webkit.org написал(а):
 
 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?
 
 The workflow is to click on the bubble to see the style errors. e.g.
 https://webkit-queues.appspot.com/results/6544662978363392
 
 
 Seems like that would require everyone to manually match errors to code 
 lines indeed, so I object against making this change for style checker 
 warnings.
 
 - WBR, Alexey Proskuryakov
 
 Yeah, seeing the style warnings as a comment (which also causes them to show 
 up in the patch review) is helpful. I was just complaining about the python 
 path spew it also includes.
 
 — Timothy Hatcher
 
 
 
 ___
 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


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

2014-01-17 Thread Alexey Proskuryakov

This has been implemented, and one unintended consequence is that this 
noticeably affects how quickly one can iterate on time sensitive patches.

It's a huge waste of time that you are no longer informed when a build fails on 
EWS. This seriously delays urgent work, as you only start working on fixes when 
you happen to manually poll, or even worse, when a reviewer tells you about 
build breakage.

It's good to not spam everyone, however patch author should be notified by EWS 
immediately I think. Some ideas:

- e-mail;
- IRC;
- browser notifications when bug page is open.

The latter might be best, as it also gives some control over whether to get 
pinged - keep the bug open if you care, close it if it's not urgent, and you 
cannot afford distraction now.

I filed https://bugs.webkit.org/show_bug.cgi?id=127203 about this.

- WBR, Alexey Proskuryakov


16 янв. 2014 г., в 15:09, Ryosuke Niwa rn...@webkit.org написал(а):

 Okay, let's remove the python paths but keep the style error messages until 
 we can improve the EWS infrastructure.
 
 - R. Niwa
 
 
 On Thu, Jan 16, 2014 at 9:41 AM, Timothy Hatcher timo...@apple.com wrote:
 On Jan 16, 2014, at 2:28 AM, Alexey Proskuryakov a...@webkit.org wrote:
 
 
 15 янв. 2014 г., в 23:02, Ryosuke Niwa rn...@webkit.org написал(а):
 
 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?
 
 The workflow is to click on the bubble to see the style errors. e.g.
 https://webkit-queues.appspot.com/results/6544662978363392
 
 
 Seems like that would require everyone to manually match errors to code 
 lines indeed, so I object against making this change for style checker 
 warnings.
 
 - WBR, Alexey Proskuryakov
 
 Yeah, seeing the style warnings as a comment (which also causes them to show 
 up in the patch review) is helpful. I was just complaining about the python 
 path spew it also includes.
 
 — Timothy Hatcher
 


___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2014-01-16 Thread Alexey Proskuryakov

15 янв. 2014 г., в 23:02, Ryosuke Niwa rn...@webkit.org написал(а):

 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?
 
 The workflow is to click on the bubble to see the style errors. e.g.
 https://webkit-queues.appspot.com/results/6544662978363392


Seems like that would require everyone to manually match errors to code lines 
indeed, so I object against making this change for style checker warnings.

- WBR, Alexey Proskuryakov___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2014-01-16 Thread Timothy Hatcher
On Jan 16, 2014, at 2:28 AM, Alexey Proskuryakov a...@webkit.org wrote:

 
 15 янв. 2014 г., в 23:02, Ryosuke Niwa rn...@webkit.org написал(а):
 
 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?
 
 The workflow is to click on the bubble to see the style errors. e.g.
 https://webkit-queues.appspot.com/results/6544662978363392
 
 
 Seems like that would require everyone to manually match errors to code lines 
 indeed, so I object against making this change for style checker warnings.
 
 - WBR, Alexey Proskuryakov

Yeah, seeing the style warnings as a comment (which also causes them to show up 
in the patch review) is helpful. I was just complaining about the python path 
spew it also includes.

— Timothy Hatcher
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2014-01-16 Thread Ryosuke Niwa
Okay, let's remove the python paths but keep the style error messages until
we can improve the EWS infrastructure.

- R. Niwa


On Thu, Jan 16, 2014 at 9:41 AM, Timothy Hatcher timo...@apple.com wrote:

 On Jan 16, 2014, at 2:28 AM, Alexey Proskuryakov a...@webkit.org wrote:


 15 янв. 2014 г., в 23:02, Ryosuke Niwa rn...@webkit.org написал(а):

  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?


 The workflow is to click on the bubble to see the style errors. e.g.
 https://webkit-queues.appspot.com/results/6544662978363392


 Seems like that would require everyone to manually match errors to code
 lines indeed, so I object against making this change for style checker
 warnings.

 - WBR, Alexey Proskuryakov


 Yeah, seeing the style warnings as a comment (which also causes them to
 show up in the patch review) is helpful. I was just complaining about the
 python path spew it also includes.

 — Timothy Hatcher

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2014-01-15 Thread Sam Weinig
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


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

2014-01-15 Thread Alexey Proskuryakov

+1

This is blocked on improving Bugzilla EWS bubbles, making it humanly possible 
to find out without a comment which tests regressed.

- WBR, Alexey Proskuryakov


15 янв. 2014 г., в 19:54, Sam Weinig wei...@apple.com написал(а):

 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


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

2014-01-15 Thread Ryosuke Niwa
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.orgbugs 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


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

2014-01-15 Thread Sam Weinig
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


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

2014-01-15 Thread Ryosuke Niwa
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


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

2014-01-15 Thread Joseph Pecoraro
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


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

2014-01-15 Thread Gyuyoung Kim
+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


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

2014-01-15 Thread 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 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


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

2014-01-15 Thread 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 rn...@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@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

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2014-01-15 Thread Ryosuke Niwa
On Wed, Jan 15, 2014 at 10:54 PM, Alexey Proskuryakov a...@webkit.org wrote:

 (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?


The workflow is to click on the bubble to see the style errors. e.g.
https://webkit-queues.appspot.com/results/6544662978363392

- R. Niwa
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev