How will you guys handle it when you get a ton of module creators start banging down your door to get a review? Will you consider bringing on others who might give reviews of the inevitably huge amount of requests you're going to get? Do you think this could potentially lead to important patch reviews as well?
Well, we're pretty much there already. At an innocent estimate, there are 12 modules now in the queue, and chx and I, both busy off our rockers, had planned to only do one a week. Oh, and I have a baby coming any day now ;) As for bringing on others, I'm ... cautious about it. On one hand, yes, the entire point of this exercise is to bring up the level of quality in contrib, and the best way to do that is to encourage more eyeballs and an attention to detail. On the other hand, I've yet to see anyone as anal as me when it comes to a code review, and I'd be ... ... cautious about posting code reviews that didn't have my level of anality on a site that largely exists because of a post I wrote complaining about code quality ;) (If I'm anal about code quality, I'm also anal about content quality). But, really, this site just started. We don't have the details worked out and, certainly, the community has some sort of say in its direction because the community is who we're writing this for. So, if anyone did want to get involved per se, definitely speak up. Note, though, that there are 2000+ modules in contrib. Even with a dozen more people reviewing at the level of anality I'd like, we're not going to change the world, or contrib, in a month. As for module creators banging down the doors, ideally, the intention of these reviews are to point out obvious, fatal, and learned, flaws, design, style, and otherwise, that every Drupal developer could benefit from. A module who is #13 in the queue is at a distinct advantage over the module at #2 in the queue, because they can address our discovered gripes in modules #1 to #12, before we get to them. That's a powerful motivator. -- Morbus Iff ( all of the above - the only checkbox worth checking ) Technical: http://www.oreillynet.com/pub/au/779 Enjoy: http://www.disobey.com/ and http://www.videounderbelly.com/ aim: akaMorbus / skype: morbusiff / icq: 2927491 / jabber.org: morbus