I came across the following articles on source control and code review:
* https://secure.phabricator.com/book/phabflavor/article/recommendations_on_revision_control/ * https://secure.phabricator.com/book/phabflavor/article/writing_reviewable_code/ * https://secure.phabricator.com/book/phabflavor/article/recommendations_on_branching/
I think everyone working on Firefox should take the time to read them as they prescribe what I perceive to be a very rational set of best practices for working with large and complex code bases.
The articles were written by a (now former) Facebooker and the recommendations are significantly influenced by Facebook's experiences. They have many of the same problems we do (size and scale of code base, hundreds of developers, etc). Some of the pieces on feature development don't translate easily, but most of the content is relevant.
I would be thrilled if we started adopting some of the recommendations such as more descriptive commit messages and many, smaller commits over fewer, complex commits.
_______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
