Skip to content

Commit 876f9be

Browse files
authored
chore: avoid work on non master branches (#53)
1 parent e2189f1 commit 876f9be

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

pkg/merge-with-label/server/base_request.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@ type BaseRequest struct {
1616
Owner struct {
1717
Login string `json:"login"`
1818
} `json:"owner"`
19-
Private bool `json:"private"`
19+
Private bool `json:"private"`
20+
DefaultBranch string `json:"default_branch"`
2021
} `json:"repository"`
2122
}
2223

pkg/merge-with-label/server/server.go

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,8 @@ func (h *Handler) handlePullRequestReview(logger *zerolog.Logger, eventID string
328328
func (h *Handler) handlePush(logger *zerolog.Logger, eventID string, body []byte, w http.ResponseWriter) {
329329
var req struct {
330330
BaseRequest
331-
Deleted bool `json:"deleted"`
331+
Deleted bool `json:"deleted"`
332+
Ref string `json:"ref"`
332333
}
333334

334335
if err := json.Unmarshal(body, &req); err != nil {
@@ -343,6 +344,14 @@ func (h *Handler) handlePush(logger *zerolog.Logger, eventID string, body []byte
343344
return
344345
}
345346

347+
if req.Ref != "refs/heads/"+req.Repository.DefaultBranch {
348+
// no need to handle pushes that are not targeted to the master branch.
349+
// pushes to branches will only be handled when there is a pull request open
350+
// and if a pull request is open, github will call the pull_request synchronize event.
351+
h.respond(w, http.StatusOK, "ok")
352+
return
353+
}
354+
346355
err := common.QueueMessage(
347356
logger,
348357
h.JetStreamContext,

0 commit comments

Comments
 (0)