From bd15d3ecb48b8a634802448671c4e14af34c47e7 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 27 Mar 2026 03:07:26 +0000 Subject: [PATCH] refactor: remove unused NewNilSigner and NewNilRSASigner functions Both functions return nil which doesn't implement the interface, making them actively misleading. The codebase already handles nil signers via nil checks throughout generators. Co-Authored-By: Claude Opus 4.6 (1M context) --- internal/signer/gpg.go | 7 +------ internal/signer/rsa.go | 7 +------ 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/internal/signer/gpg.go b/internal/signer/gpg.go index bd70e62..3262976 100644 --- a/internal/signer/gpg.go +++ b/internal/signer/gpg.go @@ -256,9 +256,4 @@ func (s *GPGSigner) GetPublicKey() ([]byte, error) { } return buf.Bytes(), nil -} - -// NewNilSigner returns a nil signer (for unsigned repositories) -func NewNilSigner() Signer { - return nil -} +} \ No newline at end of file diff --git a/internal/signer/rsa.go b/internal/signer/rsa.go index 87db276..2096504 100644 --- a/internal/signer/rsa.go +++ b/internal/signer/rsa.go @@ -120,9 +120,4 @@ func (s *AlpineRSASigner) GetPublicKey() ([]byte, error) { } return pem.EncodeToMemory(block), nil -} - -// NewNilRSASigner returns a nil RSA signer (for unsigned repositories) -func NewNilRSASigner() RSASigner { - return nil -} +} \ No newline at end of file