Update PR before trying to merge it

It probably makes sense to do things in this order rather than how we were doing it before.
palantir · Apr 7, 2021 · be21959 · be21959
1 parent c1d99af
commit be21959
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/server/handler/pull_request.go b/server/handler/pull_request.go
@@ -68,9 +68,6 @@ func (h *PullRequest) Handle(ctx context.Context, eventType, deliveryID string,
 	if err != nil {
 		return errors.Wrap(err, "failed to fetch configuration")
 	}
-	if err := h.ProcessPullRequest(ctx, pullCtx, client, config, pr); err != nil {
-		logger.Error().Err(errors.WithStack(err)).Msg("Error processing pull request")
-	}
 
 	if event.GetAction() == "labeled" || event.GetAction() == "opened" {
 		base, _ := pullCtx.Branches()
@@ -79,6 +76,10 @@ func (h *PullRequest) Handle(ctx context.Context, eventType, deliveryID string,
 		}
 	}
 
+	if err := h.ProcessPullRequest(ctx, pullCtx, client, config, pr); err != nil {
+		logger.Error().Err(errors.WithStack(err)).Msg("Error processing pull request")
+	}
+
 	return nil
 }