From 45800be002e4348e4921969b4f81842b4f587ed6 Mon Sep 17 00:00:00 2001 From: Nikola Jokic Date: Mon, 24 Nov 2025 12:05:49 +0100 Subject: [PATCH] Add error messages instead of just returning an error (#16) * Add error messages instead of just returning an error * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- config.go | 2 +- listener/listener.go | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/config.go b/config.go index 53a089a..508e174 100644 --- a/config.go +++ b/config.go @@ -32,7 +32,7 @@ type GitHubConfig struct { func ParseGitHubConfigFromURL(in string) (*GitHubConfig, error) { u, err := url.Parse(strings.Trim(in, "/")) if err != nil { - return nil, err + return nil, fmt.Errorf("failed to parse URL: %w", err) } isHosted := isHostedGitHubURL(u) diff --git a/listener/listener.go b/listener/listener.go index 3620dc3..d9d8a1b 100644 --- a/listener/listener.go +++ b/listener/listener.go @@ -76,7 +76,7 @@ func New(client *scaleset.Client, config Config) (*Listener, error) { } if err := config.Validate(); err != nil { - return nil, err + return nil, fmt.Errorf("invalid config: %w", err) } hostname, err := os.Hostname() @@ -247,7 +247,7 @@ func (l *Listener) getMessage(ctx context.Context) (*scaleset.RunnerScaleSetMess } if err := l.refreshSession(ctx); err != nil { - return nil, err + return nil, fmt.Errorf("failed to refresh message session: %w", err) } l.logger.Info("Getting next message", "lastMessageID", l.lastMessageID) @@ -284,7 +284,7 @@ func (l *Listener) deleteLastMessage(ctx context.Context) error { } if err := l.refreshSession(ctx); err != nil { - return err + return fmt.Errorf("failed to refresh message session: %w", err) } err = l.client.DeleteMessage(