BTW, I don't think the patch is necessary if there's a link to see the patch in the Web interface instead.
On Wed, Apr 21, 2010 at 8:27 PM, Jay McCarthy <[email protected]> wrote: > 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 <[email protected]> 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 <[email protected]> > Assistant Professor / Brigham Young University > http://teammccarthy.org/jay > > "The glory of God is Intelligence" - D&C 93 > -- Jay McCarthy <[email protected]> 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
