[Boost-commit] svn:boost r40760 - website/public_html/beta/community

From: daniel_james_at_[hidden]
Date: 2007-11-04 16:49:04

Author: danieljames
Date: 2007-11-04 16:49:04 EST (Sun, 04 Nov 2007)
New Revision: 40760
URL: http://svn.boost.org/trac/boost/changeset/40760

Log:
Bring the review process up to date. At least I think it is - this document is
a mess in trunk right now. Fixes #1350.

Text files modified: 
   website/public_html/beta/community/reviews.html |    58 +++++++++++++++++++++++++++++++++------ 
   1 files changed, 49 insertions(+), 9 deletions(-)

Modified: website/public_html/beta/community/reviews.html
==============================================================================
--- website/public_html/beta/community/reviews.html	(original)
+++ website/public_html/beta/community/reviews.html	2007-11-04 16:49:04 EST (Sun, 04 Nov 2007)
@@ -25,6 +25,15 @@
               <h1>Boost Formal Review Process</h1>
             </div>
 
+            <div class="admonition">
+              <h2>Before Requesting a Formal Review</h2>
+
+              <p><strong>Read and follow the Boost <a href=
+              "submission_process.htm">submission process</a>.</strong> There
+              are at least four steps a library author must take before a
+              formal review is requested.</p>
+            </div>
+
             <div class="section-body">
               <ul class="toc">
                 <li>Introduction</li>
@@ -77,6 +86,15 @@
               identify problems along the way, please note if they are minor,
               serious, or showstoppers.</p>
 
+              <p>The goal of a Boost library review is to improve the library
+              through constructive criticism, and at the end a decision must
+              be made: is the library good enough at this point to accept
+              into Boost? If not, we hope to have provided enough
+              constructive criticism for it to be improved and accepted at a
+              later time. The Serialization library is a good example of how
+              constructive criticism resulted in revisions resulting in an
+              excellent library that was accepted in its second review.</p>
+
               <p>Here are some questions you might want to answer in your
               review:</p>
 
@@ -107,6 +125,22 @@
                 comments don't obscure your overall opinion.</li>
               </ul>
 
+              <p>Many reviews include questions for library authors. Authors
+              are interested in defending their library against your
+              criticisms; otherwise they would not have brought their library
+              up for review. If you don't get a response to your question
+              quickly, be patient; if it takes too long or you don't get an
+              answer you feel is sufficient, ask again or try to rephrase the
+              question. Do remember that English is not the native language
+              for many Boosters, and that can cause misunderstandings.</p>
+
+              <p>E-mail is a poor communication medium, and even if messages
+              rarely get lost in transmission, they often get drowned in the
+              deluge of other messages. Don't assume that an unanswered
+              message means you're being ignored. Given constructively,
+              criticism will be taken better and have more positive effects,
+              and you'll get the answers you want.</p>
+
               <h2><a name="Results" id="Results"></a>Results</h2>
 
               <p>At the conclusion of the comment period, the Review Manager
@@ -168,15 +202,22 @@
                 <li>Follows review discussions regarding the library,
                 moderating or answering questions as needed.</li>
 
+                <li>Asks the review wizard for
+                permission to extend the review schedule if it appears that
+                too few reviews will be submitted during the review
+                period.</li>
+
                 <li>Decides if there is consensus to accept the library, and
                 if there are any conditions attached.</li>
 
-                <li>Posts a message on both the <strong><a href=
-                "mailto:boost_at_[hidden]">boost</a></strong> and
-                <strong><a href=
+                <li>Posts a notice of the <a href="#Results">review
+                results</a> on the regular <strong><a href=
+                "mailto:boost_at_[hidden]">boost</a></strong> mailing
+                list, the <strong><a href=
+                "mailto:boost-users_at_[hidden]">boost-users</a></strong>
+                mailing list, and the <strong><a href=
                 "mailto:boost-announce_at_[hidden]">boost-announce</a></strong>
-                lists informing members of the <a href="#Results">review
-                results</a>.</li>
+                mailing list.</li>
               </ul>
 
               <p>In other words, it is the Review Manager's responsibility to
@@ -241,10 +282,9 @@
                 major adjustments.</li>
               </ul>
 
-              <p>The role of Boost Review Wizard is currently played by
-              <a href="mailto:reportbase_at_[hidden]" class="external">Tom
-              Brinkman</a> and Ronald Garcia (garcia at cs dot indiana dot
-              edu).</p>
+              <p>The role of Boost Review Wizard is currently played by John
+              Phillips (phillips at mps dot ohio-state dot edu) and Ronald
+              Garcia (garcia at cs dot indiana dot edu).</p>
 
               <h2><a name="Fast-Track" id="Fast-Track"></a>Fast Track
               Reviews</h2>