All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Abrahams <dave@boostpro.com>
To: Jakub Narebski <jnareb@gmail.com>
Cc: Junio C Hamano <gitster@pobox.com>,
	Jonathan Nieder <jrnieder@gmail.com>,
	git@vger.kernel.org
Subject: Re: cherry-pick / pre-commit hook?
Date: Tue, 28 Dec 2010 16:00:48 -0900	[thread overview]
Message-ID: <m2bp45fl4v.wl%dave@boostpro.com> (raw)
In-Reply-To: <m3vd2djzhp.fsf@localhost.localdomain>

At Tue, 28 Dec 2010 14:38:20 -0800 (PST),
Jakub Narebski wrote:
> 
> Junio C Hamano <gitster@pobox.com> writes:
> 
> > I think the basic direction could be (I haven't thought things through,
> > just a strawman):
> > 
> >  - Allow --verify/--no-verify to all commands that possibly create a new
> >    commit, and run pre-commit hook where an updated index is about to be
> >    made into a commit (for some commands this may not be very easy);
> > 
> >  - The guideline of picking the default would probably look like this:
> > 
> >    (1) for existing commands, keep the current behaviour;
> > 
> >    (2) for a new command, --verify should be the default if the command is
> >        primarily about letting the user do what s/he would/could/should
> >        have done as "git commit" in the first place (e.g. cherry-picking
> >        one's own commit from a separate branch or rebasing one's own
> >        unpublished branch on top of updated upstream), and --no-verify
> >        otherwise (i.e. taking other's work and using it in a context
> >        different from the original).
> 
> Does it mean that for now (and perhaps also for later) it means that
> "git commit" by default runs pre-commit hook, unless one use
> --no-verify, and that all comands that create a new commit (rebase,
> cherry-pick, revert, merge/pull) can request for pre-commit hook to be
> run (if they create commit) with --verify?
> 
> I think it is a very good idea

+1!

-- 
Dave Abrahams
BoostPro Computing
http://www.boostpro.com

      reply	other threads:[~2010-12-29  1:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-08 17:10 cherry-pick / pre-commit hook? Dave Abrahams
2010-12-08 17:53 ` Jonathan Nieder
2010-12-08 21:22   ` Dave Abrahams
2010-12-08 22:05     ` Jonathan Nieder
2010-12-27  2:18       ` Dave Abrahams
2010-12-27  9:37         ` Jonathan Nieder
2010-12-27 20:58           ` Junio C Hamano
2010-12-27 21:33             ` Dave Abrahams
2010-12-28 18:16           ` Junio C Hamano
2010-12-28 22:38             ` Jakub Narebski
2010-12-29  1:00               ` Dave Abrahams [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=m2bp45fl4v.wl%dave@boostpro.com \
    --to=dave@boostpro.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=jnareb@gmail.com \
    --cc=jrnieder@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.