Code review

When there's a team of people who work collaboratively on Puppet, it is important to have good communication among its members, a common vision on how the code is organized, where logic and data are placed, and what the principles behind some design decisions are.

Many mistakes made while working on Puppet are due to incomplete knowledge of the area of effect of code changes, and this is generally due to bad communication.

For this reason, any tool that can boost communication, peer review, and discussion about code can definitely help in having a saner development environment.

Gerrit

When we work with Git, the natural companion to manage peer review and workflow authorization schemes is Gerrit. This is a web interface made by Google for the development of Android. It integrates perfectly with Git and allows commenting on any commit, vote for them, and have users that may authorize their acceptance.

Different user roles with different permission schemes and authorization workflows...