diff options
author | Dave Abrahams <dave@boostpro.com> | 2007-06-13 21:30:36 +0000 |
---|---|---|
committer | Dave Abrahams <dave@boostpro.com> | 2007-06-13 21:30:36 +0000 |
commit | 4392a76c86680f21a96a4da8bb4a2bf535b704be (patch) | |
tree | c297d42556ac44c81db0267ce8ddadeffff9b2c0 | |
parent | e46b5b463540e7de528ce0b57df7befe972992ff (diff) | |
download | boost-4392a76c86680f21a96a4da8bb4a2bf535b704be.tar.gz |
Big admonition at top about following the submission process.
[SVN r37986]
-rw-r--r-- | more/formal_review_process.htm | 482 |
1 files changed, 257 insertions, 225 deletions
diff --git a/more/formal_review_process.htm b/more/formal_review_process.htm index 4564ea4786..2f4c161d3b 100644 --- a/more/formal_review_process.htm +++ b/more/formal_review_process.htm @@ -9,281 +9,313 @@ <meta name="GENERATOR" content="Microsoft FrontPage 5.0"> <meta name="ProgId" content="FrontPage.Editor.Document"> - <title>Boost Formal Review Process</title> - </head> + <title>Boost Formal Review Process</title> +<style type="text/css"> +@import ../boost.css +.first { + margin-top: 0 } +.last { + margin-bottom: 0 } +div.attention, div.caution, div.danger, div.error, div.hint, +div.important, div.note, div.tip, div.warning, div.admonition { + margin: 2em ; + border: medium outset ; + padding: 1em } +div.attention p.admonition-title, div.caution p.admonition-title, +div.danger p.admonition-title, div.error p.admonition-title, +div.warning p.admonition-title { + color: red ; + font-weight: bold ; + font-family: sans-serif } +div.hint p.admonition-title, div.important p.admonition-title, +div.note p.admonition-title, div.tip p.admonition-title, +div.admonition p.admonition-title { + font-weight: bold ; + font-family: sans-serif } +</style> +</head> + +<body bgcolor="#FFFFFF" text="#000000"> + <table border="1" bgcolor="#007F7F" cellpadding="2"> + <tr> + <td bgcolor="#FFFFFF"><img src="../boost.png" alt= + "boost.png (6897 bytes)" width="277" height="86"></td> + + <td><a href="../index.htm"><font face="Arial" color= + "#FFFFFF"><big>Home</big></font></a></td> + + <td><a href="../libs/libraries.htm"><font face="Arial" color= + "#FFFFFF"><big>Libraries</big></font></a></td> + + <td><a href="../people/people.htm"><font face="Arial" color= + "#FFFFFF"><big>People</big></font></a></td> + + <td><a href="faq.htm"><font face="Arial" color= + "#FFFFFF"><big>FAQ</big></font></a></td> + + <td><a href="index.htm"><font face="Arial" color= + "#FFFFFF"><big>More</big></font></a></td> + </tr> + </table> + + <h1>Boost Formal Review Process</h1> + <div class="admonition-note admonition"> + <p class="first admonition-title">Before Requesting a Formal Review</p> + <p class="last"><b>Read and follow the Boost <a href= + "submission_process.htm">submission process</a>.</b> There are at + least four steps a library author must take before a formal review is + requested.</p> + </div> + + <p><a href="#Introduction">Introduction</a><br> + <a href="#Comments">What to include in Review Comments</a><br> + <a href="#Results">Results</a><br> + <a href="#Review_Manager">Notes for Review Managers</a><br> + <a href="#Submitters">Notes for Library Submitters</a><br> + <a href="#Wizard">Review Wizard</a><br> + <a href="#Fast-Track">Fast Track Reviews</a></p> + + <h2><a name="Introduction" id="Introduction">Introduction</a></h2> + + <p>Proposed libraries are accepted into Boost only after undergoing a + formal review, where Boost mailing list members comment on their evaluation + of the library.</p> + + <p>The final "accept" or "reject" decision is made by the <a href= + "#Review_Manager">Review Manager</a>, based on the review comments received + from boost mailing list members.</p> + + <p>Boost mailing list members are encouraged to submit Formal Review + comments:</p> + + <blockquote> + <ul> + <li>Publicly on the mailing list.</li> + + <li>Privately to the Review Manager.</li> + </ul> + </blockquote> + + <p>Private comments to a library submitter may be helpful to her or him, + but won't help the Review Manager reach a decision, so the other forms are + preferred.</p> + + <h2>What to include in Review <a name="Comments" id= + "Comments">Comments</a></h2> + + <p>Your comments may be brief or lengthy, but basically the Review Manager + needs your evaluation of the library. If you identify problems along + the way, please note if they are minor, serious, or showstoppers.</p> + + <p>Here are some questions you might want to answer in your review:</p> - <body bgcolor="#FFFFFF" text="#000000"> - <table border="1" bgcolor="#007F7F" cellpadding="2"> - <tr> - <td bgcolor="#FFFFFF"><img src="../boost.png" alt= - "boost.png (6897 bytes)" width="277" height="86"></td> + <ul> + <li>What is your evaluation of the design?<br></li> - <td><a href="../index.htm"><font face="Arial" color= - "#FFFFFF"><big>Home</big></font></a></td> + <li>What is your evaluation of the implementation?<br></li> - <td><a href="../libs/libraries.htm"><font face="Arial" color= - "#FFFFFF"><big>Libraries</big></font></a></td> + <li>What is your evaluation of the documentation?<br></li> - <td><a href="../people/people.htm"><font face="Arial" color= - "#FFFFFF"><big>People</big></font></a></td> + <li>What is your evaluation of the potential usefulness of the + library?<br></li> - <td><a href="faq.htm"><font face="Arial" color= - "#FFFFFF"><big>FAQ</big></font></a></td> + <li>Did you try to use the library? With what compiler? Did + you have any problems?<br></li> - <td><a href="index.htm"><font face="Arial" color= - "#FFFFFF"><big>More</big></font></a></td> - </tr> - </table> + <li>How much effort did you put into your evaluation? A glance? A quick + reading? In-depth study?<br></li> - <h1>Boost Formal Review Process</h1> + <li>Are you knowledgeable about the problem domain?</li> + </ul> - <p><a href="#Introduction">Introduction</a><br> - <a href="#Comments">What to include in Review Comments</a><br> - <a href="#Results">Results</a><br> - <a href="#Review_Manager">Notes for Review Managers</a><br> - <a href="#Submitters">Notes for Library Submitters</a><br> - <a href="#Wizard">Review Wizard</a><br> - <a href="#Fast-Track">Fast Track Reviews</a></p> + <p>And finally, every review should answer this question:<br></p> - <h2><a name="Introduction">Introduction</a></h2> + <ul> + <li>Do you think the library should be accepted as a Boost library? + Be sure to say this explicitly so that your other comments don't obscure + your overall opinion.</li> + </ul> - <p>Proposed libraries are accepted into Boost only after undergoing a - formal review, where Boost mailing list members comment on their - evaluation of the library.</p> + <h2><a name="Results" id="Results">Results</a></h2> - <p>The final "accept" or "reject" decision is made by the <a href= - "#Review_Manager">Review Manager</a>, based on the review comments - received from boost mailing list members.</p> + <p>At the conclusion of the comment period, the Review Manager will post a + message to the mailing list saying if the library has been accepted or + rejected. A rationale is also helpful, but its extent is up to the + Review Manager. If there are suggestions, or conditions that must be met + before final inclusion, they should be stated.</p> - <p>Boost mailing list members are encouraged to submit Formal Review - comments:</p> + <h2>Notes for <a name="Review_Manager" id="Review_Manager">Review + Manager</a>s</h2> + + <p>Before a library can be scheduled for formal review, an active boost + member not connected with the library submission must volunteer to be the + "Review Manager" for the library.</p> + + <p>The Review Manager:</p> + + <ul> + <li>Checks the submission to make sure it really is complete enough to + warrant formal review. See the <a href="lib_guide.htm">Boost + Library Requirements and Guidelines</a>. If necessary, work with + the submitter to verify the code compiles and runs correctly on several + compilers and platforms.</li> + + <li>Finalizes the schedule with the <a href="#Wizard">Review Wizard</a> + and the submitter .</li> + + <li>Posts a notice of the review schedule on the regular <b><a href= + "mailto:boost@lists.boost.org">boost</a></b> mailing list, the + <b><a href="mailto:boost-users@lists.boost.org">boost-users</a></b> + mailing list, and the <b><a href= + "mailto:boost-announce@lists.boost.org">boost-announce</a></b> mailing + list. - <blockquote> <ul> - <li>Publicly on the mailing list.</li> + <li>The notice should include a brief description of the library and + what it does, to let readers know if the library is one they are + interested in reviewing.</li> - <li>Privately to the Review Manager.</li> + <li>If the library is known to fail with certain compilers, please + mention them in the review notice so reviewers with those compilers + won't waste time diagnosing known problems.</li> </ul> - </blockquote> + </li> - <p>Private comments to a library submitter may be helpful to her or him, - but won't help the Review Manager reach a decision, so the other forms - are preferred.</p> + <li>Inspects the Boost <a href="../libs/libraries.htm">library + catalogue</a> for libraries which may interact with the new submission. + These potential interactions should be pointed out in the review + announcement, and the author(s) of these libraries should be privately + notified and urged to participate in the review.</li> - <h2>What to include in Review <a name="Comments">Comments</a></h2> + <li>Urges people to do reviews if they aren't forthcoming.</li> - <p>Your comments may be brief or lengthy, but basically the Review - Manager needs your evaluation of the library. If you identify - problems along the way, please note if they are minor, serious, or - showstoppers.</p> + <li>Follows review discussions regarding the library, moderating or + answering questions as needed.</li> - <p>Here are some questions you might want to answer in your review:</p> + <li>Asks the <a href="#Wizard">review wizard</a> for permission to extend + the review schedule if it appears that too few reviews will be submitted + during the review period.</li> - <ul> - <li>What is your evaluation of the design?<br> - </li> + <li>Decides if there is consensus to accept the library, and if there are + any conditions attached.</li> - <li>What is your evaluation of the implementation?<br> - </li> + <li>Posts a notice of the <a href="#Results">review results</a> on the + regular <b><a href="mailto:boost@lists.boost.org">boost</a></b> mailing + list, the <b><a href= + "mailto:boost-users@lists.boost.org">boost-users</a></b> mailing list, + and the <b><a href= + "mailto:boost-announce@lists.boost.org">boost-announce</a></b> mailing + list.</li> + </ul> - <li>What is your evaluation of the documentation?<br> - </li> + <p>In other words, it is the Review Manager's responsibility to make sure + the review process works smoothly.</p> - <li>What is your evaluation of the potential usefulness of the - library?<br> - </li> + <h2>Notes for Library <a name="Submitters" id= + "Submitters">Submitters</a></h2> - <li>Did you try to use the library? With what compiler? Did - you have any problems?<br> - </li> + <p>See <a href="submission_process.htm">Submission Process</a> for a + description of the steps a library developer goes through to get a library + accepted by Boost.</p> - <li>How much effort did you put into your evaluation? A glance? A quick - reading? In-depth study?<br> - </li> + <p>A proposed library should remain stable during the review period; it + will just confuse and irritate reviewers if there are numerous + changes. It is, however, useful to upload fixes for serious bugs + right away, particularly those which prevent reviewers from fully + evaluating the library. Post a notice of such fixes on the mailing + list.</p> - <li>Are you knowledgeable about the problem domain?</li> - </ul> + <p>Library improvements suggested by reviewers should normally be held + until after the completion of review period. If the suggested changes + might affect reviewer's judgments, post a notice of the pending change + on the mailing list.</p> - <p>And finally, every review should answer this question:<br> - </p> + <h2>Review <a name="Wizard" id="Wizard">Wizard</a></h2> - <ul> - <li>Do you think the library should be accepted as a Boost - library? Be sure to say this explicitly so that your other - comments don't obscure your overall opinion.</li> - </ul> + <p>The Review Wizard coordinates the formal review schedule:</p> - <h2><a name="Results">Results</a></h2> + <ul> + <li>Maintains a list of review manager volunteers, in the form of a + queue, so that volunteers who least recently managed reviews become the + prime candidates for upcoming reviews.</li> - <p>At the conclusion of the comment period, the Review Manager will post - a message to the mailing list saying if the library has been accepted or - rejected. A rationale is also helpful, but its extent is up to the - Review Manager. If there are suggestions, or conditions that must be met - before final inclusion, they should be stated.</p> + <li>When a formal review is requested for a library:</li> - <h2>Notes for <a name="Review_Manager">Review Manager</a>s</h2> + <li style="list-style: none"> - <p>Before a library can be scheduled for formal review, an active boost - member not connected with the library submission must volunteer to be the - "Review Manager" for the library.</p> + <ul> + <li>Assign a review manager and suggests a schedule, after checking + (via private email) availability of the volunteers at the top of + review manager queue.</li> - <p>The Review Manager:</p> + <li>Finalize the schedule, once the review manager verifies the + library is actually ready for review.</li> - <ul> - <li>Checks the submission to make sure it really is complete enough to - warrant formal review. See the <a href="lib_guide.htm">Boost - Library Requirements and Guidelines</a>. If necessary, work with - the submitter to verify the code compiles and runs correctly on several - compilers and platforms.</li> - - <li>Finalizes the schedule with the <a href="#Wizard">Review Wizard</a> - and the submitter .</li> - - <li> - Posts a notice of the review schedule on the - regular <b><a href= "mailto:boost@lists.boost.org">boost</a></b> - mailing list, - the <b><a href="mailto:boost-users@lists.boost.org">boost-users</a></b> - mailing list, and the - <b><a href="mailto:boost-announce@lists.boost.org">boost-announce</a></b> - mailing list. - - <ul> - <li>The notice should include a brief description of the library - and what it does, to let readers know if the library is one they - are interested in reviewing.</li> - - <li>If the library is known to fail with certain compilers, please - mention them in the review notice so reviewers with those compilers - won't waste time diagnosing known problems.</li> - </ul> - </li> - - <li>Inspects the Boost <a href="../libs/libraries.htm">library - catalogue</a> for libraries which may interact with the new submission. - These potential interactions should be pointed out in the review - announcement, and the author(s) of these libraries should be privately - notified and urged to participate in the review.</li> - - <li>Urges people to do reviews if they aren't forthcoming.</li> - - <li>Follows review discussions regarding the library, moderating or - answering questions as needed.</li> - - <li>Asks the <a href="#Wizard">review wizard</a> 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 notice of the <a href="#Results">review results</a> on the - regular <b><a href= "mailto:boost@lists.boost.org">boost</a></b> - mailing list, - the <b><a href="mailto:boost-users@lists.boost.org">boost-users</a></b> - mailing list, and the - <b><a href="mailto:boost-announce@lists.boost.org">boost-announce</a></b> - mailing list. - </ul> + <li>Resolve schedule slips or other issues with review managers and + submitters.</li> + </ul> + </li> - <p>In other words, it is the Review Manager's responsibility to make sure - the review process works smoothly.</p> + <li>Maintains a schedule of both past and pending reviews, in the form of + the <a href="formal_review_schedule.html">Review Schedule</a> web + page.</li> - <h2>Notes for Library <a name="Submitters">Submitters</a></h2> + <li>Resolves questions from review managers and library submitters, who + sometimes want a third opinion on questions such as "Should we extend the + review period because ...?"</li> - <p>See <a href="submission_process.htm">Submission Process</a> for a - description of the steps a library developer goes through to get a - library accepted by Boost.</p> + <li>Monitors the general review process, and makes minor adjustments as + needed, or queries the list about possible major adjustments.</li> + </ul>The role of Boost Review Wizard is currently played by <a href= + "mailto:reportbase@yahoo.com">Tom Brinkman</a> and Ronald Garcia (garcia at + cs dot indiana dot edu). - <p>A proposed library should remain stable during the review period; it - will just confuse and irritate reviewers if there are numerous - changes. It is, however, useful to upload fixes for serious bugs - right away, particularly those which prevent reviewers from fully - evaluating the library. Post a notice of such fixes on the mailing - list.</p> + <h2><a name="Fast-Track" id="Fast-Track">Fast Track Reviews</a></h2> - <p>Library improvements suggested by reviewers should normally be held - until after the completion of review period. If the suggested - changes might affect reviewer's judgments, post a notice of the - pending change on the mailing list.</p> + <p>To qualify for fast track review:</p> - <h2>Review <a name="Wizard">Wizard</a></h2> + <ul> + <li>The component must be small.</li> - <p>The Review Wizard coordinates the formal review schedule:</p> + <li>The technique must be already in use in Boost libraries and the new + component provides a common implementation.</li> - <ul> - <li>Maintains a list of review manager volunteers, in the form of a - queue, so that volunteers who least recently managed reviews become the - prime candidates for upcoming reviews.</li> + <li>A full Boost-conformant implementation is available in the + sandbox.</li> - <li>When a formal review is requested for a library:</li> + <li>The Review Wizard determines that the proposal qualifies for fast + track review.</li> + </ul> - <li style="list-style: none"> - <ul> - <li>Assign a review manager and suggests a schedule, after checking - (via private email) availability of the volunteers at the top of - review manager queue.</li> + <p>Procedure:</p> - <li>Finalize the schedule, once the review manager verifies the - library is actually ready for review.</li> + <ul> + <li>The Boost Review Wizard posts a review announcement to the main Boost + developer's list. The review period will normally last for 5 days. No two + fast track reviews will run in parallel. Fast track reviews may run + during full reviews, though generally this is to be avoided.</li> - <li>Resolve schedule slips or other issues with review managers and - submitters.</li> - </ul> - </li> + <li>After the review period ends, the submitter will post a review + summary containing proposed changes to the reviewed implementation.</li> - <li>Maintains a schedule of both past and pending reviews, in the form - of the <a href="formal_review_schedule.html">Review Schedule</a> web - page.</li> + <li>The Review Wizard will accept or reject the proposed library and + proposed changes.</li> - <li>Resolves questions from review managers and library submitters, who - sometimes want a third opinion on questions such as "Should we extend - the review period because ...?"</li> + <li>After applying the proposed changes, the component is checked into + CVS like any other library.<br> + </li> + </ul> + <hr> - <li>Monitors the general review process, and makes minor adjustments as - needed, or queries the list about possible major adjustments.</li> - </ul> - The role of Boost Review Wizard is currently played by <a href= - "mailto:reportbase@yahoo.com">Tom Brinkman</a> and Ronald Garcia - (garcia at cs dot indiana dot edu). + <p>Revised + <!--webbot bot="Timestamp" s-type="EDITED" s-format="%d %B, %Y" startspan -->15 + October, 2003<!--webbot bot="Timestamp" endspan i-checksum="38556" --></p> + <p>© Copyright Beman Dawes 2000</p> - <h2><a name="Fast-Track">Fast Track Reviews</a></h2> - <p>To qualify for fast track review:</p> - <ul> - <li>The component must be small.</li> - <li>The technique must be already in use in Boost libraries and the new - component provides a common implementation.</li> - <li>A full Boost-conformant implementation is available in the sandbox.</li> - <li>The Review Wizard determines that the proposal qualifies for fast - track review.</li> - </ul> - <p>Procedure:</p> - <ul> - <li>The Boost Review Wizard posts a review announcement to the main Boost - developer's list. The review period will normally last for 5 days. No two - fast track reviews will run in parallel. Fast track reviews may run during - full reviews, though generally this is to be avoided.</li> - <li>After the review period ends, the submitter will post a review summary - containing proposed changes to the reviewed implementation.</li> - <li>The Review Wizard will accept or reject the proposed library and - proposed changes.</li> - <li>After applying the proposed changes, the component is checked into CVS - like any other library.<br> - </li> - </ul> - <hr> - - <p>Revised - <!--webbot bot="Timestamp" s-type="EDITED" s-format="%d %B, %Y" startspan -->15 October, 2003<!--webbot bot="Timestamp" endspan i-checksum="38556" --></p> - - <p>© Copyright Beman Dawes 2000</p> - <p> - Distributed under the Boost Software License, Version 1.0. (See - accompanying file <a href="../LICENSE_1_0.txt">LICENSE_1_0.txt</a> or copy - at <a href= - "http://www.boost.org/LICENSE_1_0.txt">http://www.boost.org/LICENSE_1_0.txt</a>) - </p> - </body> + <p>Distributed under the Boost Software License, Version 1.0. (See + accompanying file <a href="../LICENSE_1_0.txt">LICENSE_1_0.txt</a> or copy + at <a href= + "http://www.boost.org/LICENSE_1_0.txt">http://www.boost.org/LICENSE_1_0.txt</a>)</p> +</body> </html> |