Reviewed by Anders
authorbeidson <beidson@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Mar 2007 19:09:15 +0000 (19:09 +0000)
committerbeidson <beidson@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Mar 2007 19:09:15 +0000 (19:09 +0000)
        Explicitly mention the prepare-Changelog step for contributing patches

        * coding/contributing.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20024 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitSite/ChangeLog
WebKitSite/coding/contributing.html

index 08784f1b93b128c547cf0b74247b80b15d8c57c2..50683004321a2fd9a359d478cae7cae1a2c345e9 100644 (file)
@@ -1,3 +1,11 @@
+2007-03-07  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Anders
+
+        Explicitly mention the prepare-Changelog step for contributing patches
+
+        * coding/contributing.html:
+
 2007-02-25  Anders Carlsson  <acarlsson@apple.com>
 
         Reviewed by Maciej.
index 9bd499aaa86c2469f283ebf31a54a679fecb8d2d..5557ea859c9749659d715e02c40d620fce739fd4 100644 (file)
@@ -19,16 +19,19 @@ without specifying the path to the script.</p>
 <p>The <tt>svn-apply</tt> and <tt>svn-unapply</tt> scripts are handy for applying patches to a tree, and rolling patches out of a tree.
 They go beyond the capabilities of the <tt>patch</tt> tool by handling files added and removed from the repository.</p>
 
+<p>Before you create your patch, you should make sure it has a Changelog entry by using the <tt>prepare-Changelog</tt> script.  It sets up a 
+template for all of the Changelog entries you will have to fill in for your patch based on everything you've changed.</p>
+
+<p>Make sure that your patch meets the
+<a href="../coding/coding-style.html">coding style guidelines</a> and has received sufficient testing.
+Bug fixes should include a <a href="../quality/testing.html">new WebKit or JavaScriptCore test</a>.</p>
+
 <p>Once you have a patch file, it must be reviewed by one of the approved WebKit reviewers.
 To request a review, attach the patch to the bug report, and mark the patch with the flag <tt>review:?</tt>. This will automatically
 send mail to <a href="http://lists.webkit.org/mailman/listinfo/webkit-reviews">webkit-reviews@lists.webkit.org</a> on your behalf. The 
 <a href="../quality/lifecycle.html">WebKit Bug Life Cycle</a> page
 has more information about the stages of a WebKit Bugzilla bug.</p>
 
-<p>Make sure that your patch includes a ChangeLog entry, meets the
-<a href="../coding/coding-style.html">coding style guidelines</a>, and has received sufficient testing.
-Bug fixes should include a <a href="../quality/testing.html">new WebKit or JavaScriptCore test</a>.</p>
-
 <p>The reviewer will typically either approve the patch (by responding with an <tt>r=me</tt> in the bug report or in e-mail
 and marking the patch <tt>review:+</tt>)
 or request revisions to the patch (and mark the patch <tt>review:-</tt>).