[Webkit-unassigned] [Bug 32542] [website] Add screenshots and clearer instructions on how to submit a patch for review

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 16 10:29:53 PST 2009


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
  Attachment #44857|review?                     |review-
               Flag|                            |

--- Comment #3 from Darin Adler <darin at apple.com>  2009-12-16 10:29:53 PST ---
(From update of attachment 44857)
Thanks for taking this on.

> +<p>WebKit uses <a href="https://bugs.webkit.org/">Bugzilla</a>

I personally don't think that using the name Bugzilla so much is helpful. I
know that's the source of the bug database, but I don't think it's the best
name to refer to the WebKit bug database -- there are lots of Bugzilla
installations. I would call it the WebKit bug database, not Bugzilla.

I feel the same way about our bug tool being named "bugzilla-tool".

> +Every contribution must have an associated bug report in Bugzilla.
> +This is true even if the contribution is not a bug fix.

I think this is wrong. Every contribution must have review. And bugs.webkit.org
is the one of the best ways to get review. But there is no rule against having
contributions go in without ever involving bugs.webkit.org as long as it is

Using bugs.webkit.org is best practice, but not required.

> +<p><img src="images/bugzilla_add_attachment.png"
> +alt='Bugzilla "Add an attachment" link' /></p>

I believe these "/>" at the end of the "<img>" tags aren't needed or helpful.

> +<p>Checking the patch checkbox and marking your patch <tt>review:?</tt>
> +are necessary for getting your patch reviewed by a WebKit reviewer.
> +Setting the review flag notifies WebKit reviewers by sending them an
> +automatic e-mail through the
> +<a href="http://lists.webkit.org/mailman/listinfo/webkit-reviews">
> +webkit-reviews</a> mailing list.
> +</p>

This implies that all reviewers subscript to the webkit-reviews mailing list,
but that is not true. Different reviewers have different ways of discovering
patches that might be reviewed.

review- for now so you can consider some of my feedback

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list