named return vals

Signed-off-by: Zach Aller <[email protected]>
argoproj-labs · Jun 14, 2024 · f4be74b · f4be74b
1 parent 0304af0
commit f4be74b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/internal/scms/fake/pullrequest.go b/internal/scms/fake/pullrequest.go
@@ -18,12 +18,12 @@ func NewFakePullRequestProvider() *PullRequest {
 	return &PullRequest{}
 }
 
-func (pr *PullRequest) Create(ctx context.Context, title, head, base, description string, pullRequest *v1alpha1.PullRequest) (string, error) {
+func (pr *PullRequest) Create(ctx context.Context, title, head, base, description string, pullRequest *v1alpha1.PullRequest) (id string, err error) {
 	pullRequest.Spec.Title = title
 	pullRequest.Spec.SourceBranch = head
 	pullRequest.Spec.TargetBranch = base
 	pullRequest.Spec.Description = description
-	err := pr.savePointer(ctx, pullRequest)
+	err = pr.savePointer(ctx, pullRequest)
 	if err != nil {
 		return "", err
 	}