feat: skip LFS hooks when pre-push hook is skipped (#818)

evilmartians · Sep 26, 2024 · 6383ff3 · 6383ff3
1 parent 6858ccb
commit 6383ff3
Showing 1 changed file with 4 additions and 4 deletions.
diff --git 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()