Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] Add .gitconfig variable commit.gpg-sign

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:56:58

Junio C Hamano [off-list ref] writes:
Joel Jacobson [off-list ref] writes:
quoted
Signed-off-by: Joel Jacobson <redacted>
---
 builtin/commit.c | 4 +++-
No docs?  No tests?

As to the design, any regular configuration variable settings must
be overridable from the command line for a single invocation. Please
design an escape hatch in, for somebody who has this configuration
variable set, but does not want to sign this commit he is about to
make.

Also do we generally use dash in the configuration variable names?
I thought the norm was section.CamelCase.
Another design issue you would need to consider before deciding to
add such a configuration is how it interacts with "git rebase -i".

The command internally runs "git commit --amend", etc. and I do not
know if _all_ users who set this configuration want to unconditionally
always re-sign all these commits.

There may be other commands that internally use "git commit" that
may be affected with such a configuration variable.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help