When I get a commit message, I want to see: the author the log message the files changed the patch
I'd like all of these things collected by push: All the log messages All the files changed The patch The "git"ty things, like updating the master branch or commit sha1s aren't useful to me. If they are their, I think they should be minimized. The sha1s could be useful if they were links to a Web interface, like github. Jay On Wed, Apr 21, 2010 at 8:10 PM, Eli Barzilay <e...@barzilay.org> wrote: > Currently, the format of the push notification emails is > > (1) a highlevel overview of the push, for example: > > mflatt has updated the `master' branch > from commit 895ca49f9c8b82fcc3f453b49e6b3938ecc1713a > to commit 5d6ae2b4b9cf00b96095cb86c68c9b61f0f573ce > > (2) a "Change Summary" section that lists all the files that were > modified/created/renamed/deleted with the number of line changes > in the combination of all of these commits > > (3) a listing of all of the commits, each one with the sha1, author, > date, log message and one-line stats > > (4) an "Overall Diff" part which has the full diff (in the form of a > patch) for all of the commits. > > Are there any comments on this layout? I can make some changes to > this, for example: > > * Have the commits listed first (swap (2) and (3)) > > * Instead of (2), proivide that information next to each commit, for > the changes in that commit > > * Also, each commit could be listed next to its own diff, instead of > one combined diff (which sounds like it can be too verbose) > > * I thought about showing URLs for the web interface, but since > they're mostly going to be the same prefix with some sha1 at the > end, maybe it should include just a pointer to where the web > interface is > > Opinions? > > -- > ((lambda (x) (x x)) (lambda (x) (x x))) Eli Barzilay: > http://barzilay.org/ Maze is Life! > _________________________________________________ > For list-related administrative tasks: > http://list.cs.brown.edu/mailman/listinfo/plt-dev > -- Jay McCarthy <j...@cs.byu.edu> Assistant Professor / Brigham Young University http://teammccarthy.org/jay "The glory of God is Intelligence" - D&C 93 _________________________________________________ For list-related administrative tasks: http://list.cs.brown.edu/mailman/listinfo/plt-dev