[plt-dev] Format of push notification emails

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 <eli at 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 <jay at cs.byu.edu>
Assistant Professor / Brigham Young University
http://teammccarthy.org/jay

"The glory of God is Intelligence" - D&C 93