From bd25a73ca5e801af4a9087597c15b1958d1c6588 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 2 Sep 2026 21:03:17 -0500 Subject: Drop unused --old flag from skiff deploy Its only caller was the post-receive hook skiff itself generates, so "hook parity" meant matching a convention we invented and fully control - there's no external contract keeping it around. deploy.Run never read it. Co-Authored-By: Claude Sonnet 5 --- cmd/skiff/main.go | 2 -- 1 file changed, 2 deletions(-) (limited to 'cmd/skiff') diff --git a/cmd/skiff/main.go b/cmd/skiff/main.go index cc48c67..52d57c1 100644 --- a/cmd/skiff/main.go +++ b/cmd/skiff/main.go @@ -59,11 +59,9 @@ func main() { func cmdDeploy(args []string) error { fs := flag.NewFlagSet("deploy", flag.ExitOnError) app := fs.String("app", "", "app name") - oldSHA := fs.String("old", "", "old commit sha (unused, kept for hook parity)") newSHA := fs.String("new", "", "new commit sha to deploy") ref := fs.String("ref", "", "ref being pushed, e.g. refs/heads/main") fs.Parse(args) - _ = oldSHA if *app == "" || *newSHA == "" { return fmt.Errorf("--app and --new are required") -- cgit v1.3