From 1df9289205e3c847e19289c54df5ad149455b518 Mon Sep 17 00:00:00 2001 From: Zach Ahn Date: Thu, 19 Sep 2024 16:08:09 -0400 Subject: [PATCH] fix: run LFS hooks after checking if hooks are skipped --- internal/lefthook/runner/runner.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/internal/lefthook/runner/runner.go b/internal/lefthook/runner/runner.go index 7579dc79..db3dce95 100644 --- a/internal/lefthook/runner/runner.go +++ b/internal/lefthook/runner/runner.go @@ -90,15 +90,15 @@ type executable interface { func (r *Runner) RunAll(ctx context.Context, sourceDirs []string) ([]Result, error) { results := make([]Result, 0, len(r.Hook.Commands)+len(r.Hook.Scripts)) - if err := r.runLFSHook(ctx); err != nil { - return results, err - } - if r.Hook.DoSkip(r.Repo.State()) { r.logSkip(r.HookName, "hook setting") return results, nil } + if err := r.runLFSHook(ctx); err != nil { + return results, err + } + if !r.DisableTTY && !r.Hook.Follow { log.StartSpinner() defer log.StopSpinner()