Re: [PATCH] Builtin-commit: show on which branch a commit was added

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



(Sorry for a late response)

On 21 sep 2008, at 12:42, Jeff King wrote:

OK, I have lived with it for a little while, and I am still annoyed. ;)

My complaints are:

1. It wastes more horizontal screen real estate, making it more likely
    that the line will wrap.

 2. In almost all of my projects (including git), I use the subject
    line convention of "subsystem: one line summary". So you end up
    with the visually confusing:

      Created commit abcd1234 on master: subsystem: one line summary

    which is even worse on a topic branch which is meaningful to the
    project:

      Created commit abcd1234 on widget: subwidget: one line summary

    which has literally left me scratching my head wondering why I put
    "widget" into the commit message.

How about something like

	Created commit abcd1234 on widget -- "subwidget: one line summary"

?
Maybe it is better to simply break the line, which solves both problems.
Something like:

I don't like a multi-line approach.. I tried it myself, and the second line
makes the first line easier to overlook

- Pieter




--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux