From 9ae3551744cb4e9aa8ba9fcca688ce55482a3b16 Mon Sep 17 00:00:00 2001 From: Yusuke Kuoka Date: Tue, 2 Mar 2021 10:55:30 +0900 Subject: [PATCH] Remove unnecessary GitHub API calls (#363) The controller had the 2 extra and redundant calls to List Workflow Runs API. Ref #362 --- github/github.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/github/github.go b/github/github.go index e1f6dc8e..5ab3f4af 100644 --- a/github/github.go +++ b/github/github.go @@ -211,8 +211,6 @@ func (c *Client) listRunners(ctx context.Context, enterprise, org, repo string, } func (c *Client) ListRepositoryWorkflowRuns(ctx context.Context, user string, repoName string) ([]*github.WorkflowRun, error) { - c.Client.Actions.ListRepositoryWorkflowRuns(ctx, user, repoName, nil) - queued, err := c.listRepositoryWorkflowRuns(ctx, user, repoName, "queued") if err != nil { return nil, fmt.Errorf("listing queued workflow runs: %w", err) @@ -232,8 +230,6 @@ func (c *Client) ListRepositoryWorkflowRuns(ctx context.Context, user string, re } func (c *Client) listRepositoryWorkflowRuns(ctx context.Context, user string, repoName, status string) ([]*github.WorkflowRun, error) { - c.Client.Actions.ListRepositoryWorkflowRuns(ctx, user, repoName, nil) - var workflowRuns []*github.WorkflowRun opts := github.ListWorkflowRunsOptions{