ab/config-based-hooks-2 (was: What's cooking in git.git (Nov 2021, #07; Mon, 29))
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Nov 30, 2021, 21:12 UTC
- Message-ID
- <211130.86bl21cpkd.gmgdl@evledraar.gmail.com>
- In-Reply-To
- <xmqqzgpm2xrd.fsf@gitster.g>
On Mon, Nov 29 2021, Junio C Hamano wrote:
Show 21 quoted lines
> * ab/config-based-hooks-2 (2021-11-24) 18 commits
> - run-command: remove old run_hook_{le,ve}() hook API
> - receive-pack: convert push-to-checkout hook to hook.h
> - read-cache: convert post-index-change to use hook.h
> - commit: convert {pre-commit,prepare-commit-msg} hook to hook.h
> - git-p4: use 'git hook' to run hooks
> - send-email: use 'git hook run' for 'sendemail-validate'
> - git hook run: add an --ignore-missing flag
> - hooks: convert worktree 'post-checkout' hook to hook library
> - hooks: convert non-worktree 'post-checkout' hook to hook library
> - merge: convert post-merge to use hook.h
> - am: convert applypatch-msg to use hook.h
> - rebase: convert pre-rebase to use hook.h
> - hook API: add a run_hooks_l() wrapper
> - am: convert {pre,post}-applypatch to use hook.h
> - gc: use hook library for pre-auto-gc hook
> - hook API: add a run_hooks() wrapper
> - hook: add 'run' subcommand
> - Merge branch 'ab/config-based-hooks-1' into ab/config-based-hooks-2
>
> More "config-based hooks".I know Emily's keen to get to the conclusion of the "config-based-hooks" story. I think it's gotten a lot of careful review already, and it would be nice to have it land soon in this cycle so the follow-up topics can be queued after it.
I re-rolled it a ~week ago[1], but that was a trivial change to save you from dealing with a compilation error as it got merged with my own "ab/run-command" (it was using "env", not "env_array")[1]. Other than that there's been no change to it for the last month since the v4[2].
1. https://lore.kernel.org/git/cover-v5-00.17-00000000000-20211123T114206Z-avarab@gmail.com/ 2. https://lore.kernel.org/git/cover-v4-00.17-00000000000-20211101T184938Z-avarab@gmail.com/