On Tue, Aug 03, 2021 at 09:38:30PM +0200, Ævar Arnfjörð Bjarmason wrote: The subject is now a little confusing (and I guess it would have been before, too). Can we remove "this new library" and replace it with something that makes sense on its own? > > Move the find_hook() function from run-command.c to a new hook.c > library. This change establishes a stub library that's pretty > pointless right now, but will see much wider use with Emily Shaffer's > upcoming "configuration-based hooks" series. > > Eventually all the hook related code will live in hook.[ch]. Let's > start that process by moving the simple find_hook() function over > as-is. > > Signed-off-by: Emily Shaffer <emilyshaffer@xxxxxxxxxx> > Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> It doesn't make sense for me to add a Reviewed-by line here, I guess. But it seems to me to be fine to do this early on instead of at the very end. - Emily
