Skip to content

Commit 40c6cda

Browse files
authored
Merge pull request #74 from vitessio/fetch-by-prnum
Fetch by PR number rather than ref
2 parents a3bd3d3 + ef4114f commit 40c6cda

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

go/pull_request.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -609,8 +609,8 @@ func (h *PullRequestHandler) createCobraDocsPreviewPR(
609609

610610
remote := pr.GetBase().GetRepo().GetCloneURL()
611611
ref := pr.GetBase().GetRef()
612-
if err := vitess.FetchRef(ctx, remote, ref); err != nil {
613-
return nil, errors.Wrapf(err, "Failed to fetch %s:%s to %s for %s", remote, ref, op, pr.GetHTMLURL())
612+
if err := vitess.FetchRef(ctx, "origin", fmt.Sprintf("refs/pull/%d/head", pr.GetNumber())); err != nil {
613+
return nil, errors.Wrapf(err, "Failed to fetch Pull Request %s/%s#%d to %s for %s", vitess.Owner, vitess.Name, pr.GetNumber(), op, pr.GetHTMLURL())
614614
}
615615

616616
if err := vitess.Checkout(ctx, "FETCH_HEAD"); err != nil {

0 commit comments

Comments
 (0)