On Fri, Jun 01, 2018 at 03:06:10AM -0400, Jeff King wrote:
> On Mon, May 28, 2018 at 07:56:12PM +0200, Ævar Arnfjörð Bjarmason wrote:
>
> >
> > On Mon, May 28 2018, Robert P. J. Day wrote:
> >
> > > (apologies for more pedantic nitpickery, just little things i'm
> > > running across in my travels. aside: i actually teach git courses, so
> > > it's a bit embarrassing that i don't know some of this stuff. *sigh*.)
> >
> > Aside from maybe the empty branches/ directory (see c8a58ac5a5 ("Revert
> > "Don't create the $GIT_DIR/branches directory on init"", 2009-10-31)),
> > none of this is needed.
> >
> > I wish we didn't create any of this stuff, but have never been inclined
> > to make that my hill to die on.
> >
> > I think we're much better off just shipping e.g. a single README file in
> > hooks/, or just nothing at all.
>
> FWIW, that's my opinion, too (including the "hill to die on" part).
>
+1. It'd probably be worth having the sample hooks be part of the
installation, but not as part of every repository (e.g., hold them under
/usr/share/git/samples/hooks/ or something along those lines).
> I also wish hooks were just shell snippets in the config files that
> could follow the usual config-precedence rules.
I like this idea, but I'd probably keep the snippets in a separate file
to keep things clean.
Thanks,
-Santiago.
Attachment:
signature.asc
Description: PGP signature
