Reviews and mentoring for Swarnendu-Dutta

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 Swarnendu-Dutta on February 11, 2015 at 3:08pm
Last updated by Swarnendu-Dutta on Thu, 2015-02-26 18:31

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

Project Reviews

1. [D7] InviteReferrals - Referral Marketing Software
2. [D7] Youtube Video Uploader
3. [D7] Role expose
4. [D7] Menu Management Languages
5. [D7] User Recording
6. [D7] Node title help text
7. [D7] Menu block ignored links

I will keep on reviewing other projects. Any help from the expert reviewers or anyone would be appreciated.

Thanks,
Swarnendu Dutta

Comments

Thanks For starting

Posted by naveenvalecha on February 11, 2015 at 3:35pm
naveenvalecha's picture

Thanks Swarnendu for starting.I hope you would also encourage others guys(whom I met at 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: