nontrivial patches should be reviewed before committing. depending on the content of the patch, this could be as simple as getting someone on irc to do a quick sanity-check, or it could turn into a full discussion about the pros and cons of various ways of implementing something. :) we have a review board to post patches to (sadly it only works in firefox), and a script to easily post patches. the original script and instructions for using it are on aaron's blog. note that the reviewboard url and path have changed since that post; here's an up-to-date sample .reviewboardrc. if you're using git-svn, you will want the updated script, plus this script for producing compatible diffs.