On Thu, Mar 09, 2017 at 02:46:53PM -0500, Ehsan Akhgari wrote: > I review a large number of patches on a typical day, and usually I have to > spend a fair amount of time to just understand what the patch is doing. As > the patch author, you can do a lot to help make this easier by *writing > better commit messages*. Starting now, I'm going to try out a new practice > for a while: I'm going to first review the commit message of all patches, > and if I can't understand what the patch does by reading the commit message > before reading any of the code, I'll r- and ask for another version of the > patch.
Sometimes, the commit message does explain what it does in a sufficient manner, but finding out why requires reading the bug, assuming it's written there. I think this information should also be in the commit message. Mike _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform