On Monday, August 15, 2011 20:52:46 Jacob Carlborg wrote: > On 2011-08-15 20:19, dsimcha wrote: > > I say give Jonas one more day to post the official review request for > > the Curl wrapper, and if he doesn't then we can move CSV parser ahead. > > > > 1. We've got an embarrassment of riches with how long the review queue > > is getting. It's a shame to have so many bubbles in it. > > > > 2. If Jonas is busy/out of town/otherwise not reading the forums, then > > this isn't a good time to review the curl wrapper anyhow. > > > > 3. We'd only be pushing the Curl wrapper back one spot, to behind the > > CSV parser, so it's not like we'd be severely devaluing it. > > How bad would it be to review more than one module?
Andrei is against it because it divides people's attention. There are only so many eyeballs, and if you review multiple items at once, one or both suffer from not getting as much review as they would have were they to be reviewed separately. - Jonathan M Davis
