[webkit-reviews] review denied: [Bug 173667] Sorted EWS on Patch Page : [Attachment 315850] Added Individual EWS status pages and kept the Original EWS Page.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jul 31 14:17:53 PDT 2017


Aakash Jain <aakash_jain at apple.com> has denied	review:
Bug 173667: Sorted EWS on Patch Page
https://bugs.webkit.org/show_bug.cgi?id=173667

Attachment 315850: Added Individual EWS status pages and kept the Original EWS
Page.

https://bugs.webkit.org/attachment.cgi?id=315850&action=review




--- Comment #5 from Aakash Jain <aakash_jain at apple.com> ---
Comment on attachment 315850
  --> https://bugs.webkit.org/attachment.cgi?id=315850
Added Individual EWS status pages and kept the Original EWS Page.

This patch does a lot of things which are totally unrelated to sorting the
status messages. Please have the clean patch doing only what the bug is about.
Also when you upload a new patch, please mark this patch as obsolete.

Also, you should not include your local changes you use for debugging in the
patch (e.g.: common/net/bugzilla/bugzilla.py).


More information about the webkit-reviews mailing list