Skip to content

make --hook-dir optional for hook-impl#3662

Merged
asottile merged 1 commit intomainfrom
hook-impl-optional-hook-dir
Apr 21, 2026
Merged

make --hook-dir optional for hook-impl#3662
asottile merged 1 commit intomainfrom
hook-impl-optional-hook-dir

Conversation

@asottile
Copy link
Copy Markdown
Member

for git 2.54+ hooks it does not make sense to run legacy hooks (they will be run anyway at the original path and/or multiple commands can be configured for a single hook)

in that case we skip running them via hook-impl by not passing --hook-dir at all

@asottile asottile enabled auto-merge April 21, 2026 19:49
@asottile asottile merged commit 2a51ffc into main Apr 21, 2026
11 checks passed
@asottile asottile deleted the hook-impl-optional-hook-dir branch April 21, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant