From 469438ebc558143ed56cbcf981e9b7dea1cc9de0 Mon Sep 17 00:00:00 2001 From: Kynan Ware <47394200+BagToad@users.noreply.github.com> Date: Wed, 6 May 2026 15:52:21 -0600 Subject: [PATCH] Remove redundant nil check in IssueNodeID GraphQL already returns an error when the repository or issue does not exist, making the empty-ID check unreachable. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- api/queries_issue.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/api/queries_issue.go b/api/queries_issue.go index d9cf73be1..1cd5549a4 100644 --- a/api/queries_issue.go +++ b/api/queries_issue.go @@ -642,8 +642,5 @@ func IssueNodeID(client *Client, repo ghrepo.Interface, number int) (string, err if err != nil { return "", err } - if result.Repository.Issue.ID == "" { - return "", fmt.Errorf("issue #%d not found in %s", number, ghrepo.FullName(repo)) - } return result.Repository.Issue.ID, nil }