On Thu, Jun 12, 2014 at 02:38:58PM -0500, Caleb Thompson wrote:
> This patch allows people to set commit.verbose to implicitly send
> --verbose to git-commit.
>
> This version incorporates changes suggested by Eric Sunshine, Duy
> Nguyen, and Jeremiah Mahler.
>
> It introduces several cleanup patches to t/t7505-commit-verbose.sh to
> bring it closer to the current state of the tests as Eric has explained
> them to me, then adds the verbose config and --no-verbose flag.
>
> Since the last version of this patch
> (http://marc.info/?l=git&m=140251155830422&w=2), I've made the following
> changes:
>
> * Revert change to flags, as --no-verbose already existed and worked as
> expected with the commit.verbose configuration. Thanks to René Scharfe.
> * Fix <<-'EOS' style for check-for-no-diff script. Thanks to Mike Burns.
>
> Additionally, this set of patches was generated by format-patch, so it
> should work correctly with git-am.
>
> ------------------------------------------------------
>
> Caleb Thompson (4):
> commit test: Use test_config instead of git-config
> commit test: Use write_script
> commit test: test_set_editor in each test
> commit: add commit.verbose configuration
>
> Documentation/config.txt | 5 +++
> Documentation/git-commit.txt | 8 ++++-
> builtin/commit.c | 4 +++
> contrib/completion/git-completion.bash | 1 +
> t/t7507-commit-verbose.sh | 64
> +++++++++++++++++++++++++---------
> 5 files changed, 64 insertions(+), 18 deletions(-)
>
> --
> 2.0.0
>
The patches look good, they apply clean ('git am'), and all tests pass.
Reviewed-by: Jeremiah Mahler <[email protected]>
--
Jeremiah Mahler
[email protected]
http://github.com/jmahler
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
