[phobos] std.log ready for review!
David Nadlinger
code at klickverbot.at
Mon Feb 13 09:11:05 PST 2012
Yep, I updated Trello as well now – just wanted to make sure that nobody
on the list was already expecting to see another module reviewed first, etc.
As for concurrent reviews, I'm not sure what is the better choice, for
the obvious reasons (diluting attention vs. avoiding long wits for
review). If the rate of Phobos submissions will go up, the question will
certainly become more important.
David
On 2/13/12 2:55 PM, Steve Schveighoffer wrote:
> If you want to be the review manager, just drag your icon to the item you want to manage in trello (I saw you joined the board). Maybe also put a comment indicating that you are the review manager (there appears to be no way to designate a role for someone in a particular card). Then when you wish to announce to the NG, just move it to the "In Review" list.
>
> Note that the ordering of lists is pertinent, meaning they aren't in just some random order. Any new cards added to the ready to review queue should go at the bottom. I arbitrarily chose the order when I created the std.log and std.uuid cards, so I think it's fine if they got reordered.
>
> I think the agreement was that we would only review one item at a time, but I don't know if that was a mandate from Andrei or a consensus. Is that what should happen or should we allow more than one review at a time?
>
>
> -Steve
More information about the phobos
mailing list