Review

[Jonathan M Davis]

On 2011-07-05 14:26, jdrewsen wrote:
> Hi,
> 
>     I believe the curl wrapper is ready for a review now. Is there
> anything else in the review queue or can we schedule the curl wrapper now?

Robert Jacques has some changes to std.variant which are required for his 
changes to std.json which he was going to put up for review, but I guess that 
he hasn't had the time to finish that up yet, so we can't review them yet. 
std.path and std.log are on hold because their authors were going on vacation, 
and apparently std.benchmark needs a bit of further work on it now that it's 
become its own module, and Andrei hasn't had time to complete that yet. So, 
nothing is actually up for review just yet.

That being the case, I think that we can review the curl wrapper. I guess that 
I can be the review manager for it, since someone needs to step up and take 
that role for it. How about we review it until the 25th and then reassess 
where it stands? That's a little over 2 weeks. If it needs more review at that 
point, we can extend the review time (or put it on the backburner if it turns 
out that you need more time to iron out changes and something else is ready 
for review by then). Or if it actually looks like it's been amply reviewed and 
updated, then we can set up a vote. Does that sound good to you?

- Jonathan M Davis