Guard installs when hooks path is external#232
Conversation
|
Thanks @nshkrdotcom The guard against an external hooksPath is worth having. But I have two asks before this can be merged:
|
e7d0657 to
e0a354c
Compare
|
Thanks @qgadrian, it's updated. I rebased on the latest Linked worktrees and submodules are allowed by default: the guard trusts hooks paths inside Git's common directory, while arbitrary external Everything passes locally on Elixir 1.20.2 / OTP 29: 55 tests, formatting, strict Credo, warnings-as-errors compilation, and Dialyzer. The GitHub Actions run is waiting for your approval. I haven't put this through the paces, so it will certainly need your detailed review. The implementation itself was done with |
|
@nshkrdotcom I would appreciate if you can resolve the conflicts on this PR 🙏 Also, it is not enforced yet at project level, but a commit message stating the model/effort used would also be helpful. |
Summary
allow_external_hooks_pathconfig +GIT_HOOKS_ALLOW_EXTERNALenv override.Testing