cr0ss [cr0ss]

Please change only the status, when you upload new code; other metadata are not thought to be changed by the applicant.

IMO, You should really provide this as a default view for Views module instead of a hook_block implementation. But that's not necessarily a reason to reject this application.

If there's a compeling reason to use hook_block, then you should use $op = 'configure' instead of providing a separate config page through hook_menu.

Top Commentators module could be expanded with Gravatar or other modules integration

The same is true for Views, which offers integration support for third-party modules. The number of modules that already offer integration with Views proves that.

I would mark this RTBC once hook_block('configure') is used, if everything else checks out.

If you want to be real clever and serve both audiences, you could wrap your hook_block code in a if (!module_exists('views')) . But again, I wouldn't consider that a requirement.

Don't include License.txt. Drupal.org add this file to packages automatically, since all code is required to be GPL v2+.

You need to check_plain($user->name) before output, for example, in _top_commentators_get_list() when not using l(). This is a security issue, and the last thing holding back a green light, IMO.

Also, the indentation should match the code style guidelines. (Only 2 spaces, always), but based on discussions yesterday, I don't think we're going to hold back CVS accounts based on style any more.

A quick check of the forums shows that there are a number of users looking for a simple solution like this, so this module would be a welcome addition to the repository.

Looks good to me.

    $user_count = ($show_number)?' (' . $user->count . ')':'';
    if(empty($user->homepage) || !$link_homepage)
      $items[] = check_plain($user->name) . $user_count;
    else
      $items[] = l($user->name, $user->homepage) . $user_count;

There should be a space after if, and the curly brackets are missing.

There is something that you missed, it seems.
The project page is created from the menu Create content, and then selecting Project; the creation of the project page is independent from the creation of the directory in CVS, and it is suggested to create first the project page (see http://drupal.org/handbook/cvs/projects).