[PATCH 1/1] devel/STYLE: information how to enable standard pre-commit hook
Jani Nikula
jani at nikula.org
Tue Mar 26 02:18:11 PDT 2013
On Sat, 23 Mar 2013, Tomi Ollila <tomi.ollila at iki.fi> wrote:
> It is easier to enable git standard pre-commit hook, when the
> operation to do so is presented.
> ---
> devel/STYLE | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/devel/STYLE b/devel/STYLE
> index 0792ba1..92de42c 100644
> --- a/devel/STYLE
> +++ b/devel/STYLE
> @@ -45,8 +45,9 @@ function (param_type param, param_type param)
> - likewise, there is a space following keywords such as if and while
> - every binary operator should have space on either side.
>
> -* No trailing whitespace. Please enable the standard pre-commit hook
> - in git (or an equivalent hook).
> +* No trailing whitespace. Please enable the standard pre-commit hook in git
> + (or an equivalent hook). The standard pre-commit hook is enabled by simply
> + renaming file '.git/hooks/pre-commit.sample' to '.git/hooks/pre-commit' .
<bikeshedding>
Or just refer to http://git-scm.com/book/en/Customizing-Git-Git-Hooks or
man githooks if you really need to. Older versions of git used to have
properly named hooks with the executable bit unset...
Personally I wouldn't ask anyone to enable the hook because I wouldn't
enable it in the first place. There's $EDITOR and 'git diff --check' to
do that kind of thing... When I want to commit I want to commit, warts
and all, and I don't want a hook to prevent me from doing so!
</bikeshedding>
BR,
Jani.
More information about the notmuch
mailing list