[Webkit-unassigned] [Bug 152345] Teach dashboard code to compare non-integer revisions.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=152345

Daniel Bates <dbates at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #268424|review?, commit-queue?      |review-, commit-queue-
              Flags|                            |

--- Comment #17 from Daniel Bates <dbates at webkit.org> ---
Comment on attachment 268424
  --> https://bugs.webkit.org/attachment.cgi?id=268424
1. Adding QUnit for unit testing.

Please file a new bug to add QUnit that blocks this bug (bug #152345) unless you plan to address the original purpose of this bug as described in comment #0 in the short term (at most a week).

It is preferred to break a patch into dependent bugs instead of separating a patch into sub-patches associated with the same bug because it makes it keeps each bug focused on exactly change. Among other benefits, using the one patch per bug strategy allows for having a dedicated forum for talking about the patch that may be otherwise hard to follow (or considered noise to some people) when interleaved in a bigger conversation about a bug that has multiple patches. In the rare cases that its reasonable to have one bug with multiple patches (parts) and each part lands as they are written then we tend to format the ChangeLog entry with the same bug title and bug URL for each of these parts and mention in the description portion of the ChangeLog entry for each patch that it represents a part of the bug fix. One such example of this can be seen in the ChangeLog entries in the patches on bug #136516 that landed in <http://trac.webkit.org/changeset/173260> and <http://trac.webkit.org/change

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160108/3829f664/attachment.html>