Reviews and Mentoring for Sagar Ramgade

Events happening in the community are now at Drupal community events on www.drupal.org.
You are viewing a wiki page. You are welcome to join the group and then edit it. Be bold!
Posted by sagar ramgade on February 11, 2015 at 2:10pm
Last updated by sagar ramgade on Wed, 2015-02-11 14:10

This wiki page is created to keep track of Sagar Ramgade's manual reviews of projects in the project application queue. As a reviewer I hope to get some advice from more experienced reviewers. Please use the comment field to add your advice, corrections, and observations.

Reviews List:

Full Project Promote Issues:

  1. [D7] IXI: XML Importer
  2. [D7] World of Warcraft Tools
  3. [D7] User Navigation
  4. [D7] Product Specification
  5. [D7] Drop Down Login
  6. [D7] Favicon Generator
  7. [D7] Site Info
  8. [D7] Ubercart Veritrans

Comments

Thanks For starting this

Posted by naveenvalecha on February 11, 2015 at 2:38pm
naveenvalecha's picture

Thanks Sagar for starting.I hope you would also encourage others guys at the DCM to review the project applications and help to reduce the backlog project applications that are under review https://www.drupal.org/project/issues/projectapplications?text=&status=8...

By @mpdonadio from my mentoring:

The main application blockers are security problems, licensing issues, and third-party code. The fourth are "major API problems". To quote @klausi from my mentoring:

Project application reviews are basically sanity checks, so while providing all details what can be improved is very valuable we have to consider what issues are really blocking approval. If we are confident that the people basically know what they are doing then we should not hold them back. With the exception of security issues and licensing issues.

I personally set applications to needs work if the API usage is so painfully wrong or the spaghetti code issues just keep piling up. Fortunately people fix all the things from pareview.sh, so at least we don't have to complain about coding standards anymore.

Naveen Valecha
https://www.valechatech.net

Code review for security advisory coverage applications

Group organizers

Group notifications

This group offers an RSS feed. Or subscribe to these personalized, sitewide feeds: