Code Review of Full Project Applications | Home
Our Mission
To discuss, document, and rally around the Full Project code review process for new contributors to Drupal. To help people become (better) code reviewers so that the barrier to code contributions on the Drupal infrastructure is minimal but still valuable and to enhance the code on Drupal.org overall.
Started from webchick's talk at DrupalCon Copenhagen and then further enhancements. This is NOT a place to ask someone to review your application. Please read the instructions and use the full project instructions to apply for access.
Our Community
The Code Review team is dedicating to being a strong, cohesive force in bringing in new code contributors to Drupal.org.
- List of Reviewers
- List of Review Administrators
- Team Leads: zzolo
- On IRC: #drupal-codereview
Hold a Sprint!
Code Review Sprints are a great way to build community and get reviews done.
Process Building
It is important to define and document the process of reviewing code as it is so important to the community.
Go Ahead and Review
Anyone with some basic module writing skills can do reviews.
Storytelling and Feedback
It is important to get feedback about this process.
TODO: Create a way for anonymous feedback.
Recent Content
- Not a a serious applicant?
- Denver Project Application Sprint Brainstorming session - Time to start planning
- Third party API's or Native Jquery API?
- How much should we demand of the applicants?
- Bug squad feature
- Project Apps State of the Union
- Refactoring of other Drupal 6 module which is not on d.o
- Shared accounts?
- Too short?
- question about a code review for the admins here
- Getting code sniffer to work in eclipse with .module, .inc, and .info files
- Another question on a review
- Question about doing a code review
- Encouraging applicants to review other projects
- Applicants with Multiple Applications
- "Lost" Contributors
- How shall we handle non-typical/highly advanced applications?
- New 'Project Applications' d.o development environment
- How to seach text from .txt file
- How much code do we need to approve a user?
- Applicants' motivation analysis and possible conclusions
- Code standards review is a incredibly difficult - http://ventral.org/pareview should be set as the standard
- Developer / Project Manager of Web Solutions | University of Arizona
- PAReview.sh - online service
- Trouble with newlines
- Klausi in Community Spotlight on front page
- Do we approve people with "Features" only projects?
- Project Application Security Review Mentoring
- things to automate in the code review
- Increasing efficiency in manual code reviews
- Feedback for reviewers?
- Just 100 more
- Meta Discussion: Project Application Process Revamp
- Reply before even looking at the code?
- What are the most common problems you spot in applications?
- Kicking it old Skool!
- Most active commenters in post git process in the last 30 days, 90 days, all time
- Project Applications which require Core edits - Is this a blocker?
- Evolution of the Project Application Process - 'Coles Notes' summary
- Evolution of the Project Application Process (Part 4) - Sanity Check - Wait a Minute Here ...
- Evolution of the Project Application Process (Part 3) - The Proposal
- Evolution of the Project Application Process (Part 2) - Where the Existing Process Breaks
- Evolution of the Project Application Process (Part 1) - Today's Project Application Process
- Best practice: code review in application issue or projects sandbox?
- Automated Coder Reviews for New Project Applications - Planning/Development
- Current Applications by Age (as of June 25th)
- 'Request a Second Opinion' Here!
- Get a second opinion
- Solutions (got tired of looking at that title... thanks sreynen)
- Automation of various Code Review steps
