Re: [apps-review] Apps review team review request template

[SM <[e-mail]>]

Hi Barry,
At 08:18 23-09-2011, Barry Leiba wrote:
>Seeing how many reviewers post their reviews to apps-review prompts me
>to suggest a change to the up-front paragraph of your review request
>template.  I suggest this (example taken from a recent review
>request):

Thanks for the thoughtful comments.  I'll use your new text.

As a general comment, we send the review to apps-discuss as it has a 
wider readership.

>--------------------------------------------------
>OLD
>The review should be sent to apps-discuss, the authors, the IESG, the
>WG Chairs and document shepherd, if applicable.
>
>NEW
>The review should be sent to apps-discuss, the IESG, the authors, the
>WG Chairs and document shepherd, if applicable.  You can use the tools
>alias [e-mail] to cover the authors,
>chairs, and shepherd.  If your review recommends significant changes
>to a working-group document, you should also consider copying the
>working group's mailing list.

Ok.

General comment, some of the reviews are early reviews, i.e. they are 
performed before a Last Call has been initiated.  It is easier to ask 
for a change and less work for the author/working group.  I don't ask 
for a Cc to the IESG in such cases.  If the Apps ADs or anyone thinks 
that it is appropriate to also copy the IESG in such cases, please let me know.

I would like to ask you whether it is worth picking editorial nits in 
an early review.

>Suggested distribution:
>    To: [e-mail], [e-mail]
>    cc: [e-mail]
>--------------------------------------------------
>
>
>I think that makes it easier for the reviewers to get it right, and
>not all the reviewers know how to use the tools aliases, so I think
>this will help.  Want to try it?

I will try that.

>I would also update the wiki, but I can't see how to (can you tell
>me?).  If you want to, here's my suggestion for

Please ask Ned for a login for http://www.apps.ietf.org/user/  If 
anyone needs a login, feel free to email me.

>http://www.apps.ietf.org/content/apps-review-template
>
>--------------------------------------------------
>OLD
>This page provides a template for reviews provided by members of the
>Apps-Review Team.
>Reviewers: Please send all reviews to the apps-discuss list and make
>sure to cc the author(s) of the specification you are reviewing! You
>might also want to check out some samples, such as the reviews of
>draft-wahl-ldap-p3p, draft-ietf-btns-c-api,
>draft-ietf-calsify-2446bis, draft-merrick-jms-uri, and
>draft-dusseault-http-patch-15.
>
>NEW
>This page provides a template for reviews provided by members of the
>Apps-Review Team.
>
>Reviewers: Please send all reviews to the apps-discuss list
>([bold]not[/bold] to the apps-review list), and to the ".all" tools
>alias for the specification you are reviewing (see below)!  In cases
>where your review recommends significant changes to a working-group
>document, you should also consider copying the working group's mailing
>list.
>
>You might also want to check out some samples, such as the reviews of
>draft-wahl-ldap-p3p, draft-ietf-btns-c-api,
>draft-ietf-calsify-2446bis, draft-merrick-jms-uri, and
>draft-dusseault-http-patch-15.
>
>Suggested distribution list for reviews:
>    To: [e-mail], 
> [e-mail]
>    cc: [e-mail]
>--------------------------------------------------

That looks good.  I posted your change.  Feel free to edit it.

Should the examples be updated?

BTW, if you have notice any drafts from other areas that might 
benefit from an early review, please let me know.

Best regards,
-sm