From: Jeff King <peff@peff.net>
To: Sebastian Schuberth <sschuberth@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: git commit --cleanup and templates
Date: Wed, 27 Jan 2010 06:48:20 -0500 [thread overview]
Message-ID: <20100127114820.GA12656@coredump.intra.peff.net> (raw)
In-Reply-To: <hj7bh9$adp$1@ger.gmane.org>
On Wed, Jan 20, 2010 at 05:37:47PM +0100, Sebastian Schuberth wrote:
> when I recently for the first time used commit templates, I was
> surprised to see that git by default strips leading empty lines that
> I deliberately added to the template file when opening the template
> in the editor. I understand that I could use a different cleanup
> mode, but in fact I *want* git to clean everything, but only *after*
> I finished editing the commit message, not already when opening the
> editor.
>
> Before submitting a patch, I'd like to hear what other people think
> about this, or whether I'm overseeing something.
Nobody has responded, so I doubt anybody has strong objections (or maybe
they just didn't read your message. :) ).
Personally, the behavior you are proposing makes more sense to me, and I
don't think you will really be hurting anybody by changing it. You will
not affect the final commit message, but only what the user sees in
their editor. And if people are really attached to not having extra
whitespace in the editor, then they can simply clean up their templates.
So I would suggest working on a patch if it is something you want.
-Peff
prev parent reply other threads:[~2010-01-27 11:48 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-20 16:37 git commit --cleanup and templates Sebastian Schuberth
2010-01-27 11:48 ` Jeff King [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100127114820.GA12656@coredump.intra.peff.net \
--to=peff@peff.net \
--cc=git@vger.kernel.org \
--cc=sschuberth@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).