aboutsummaryrefslogtreecommitdiffstats
path: root/src/pull_request.rs
diff options
context:
space:
mode:
authorTeddy Wing2017-11-17 01:08:55 +0100
committerTeddy Wing2017-11-17 01:08:55 +0100
commit399e04a697e42c9a684f14349f6d94208e719832 (patch)
treea2335f1778d8d99e82c233dcdbbeea21167e948d /src/pull_request.rs
parent205fc3b71bd44067685c8517e7d126e3574827e1 (diff)
downloadkipper-399e04a697e42c9a684f14349f6d94208e719832.tar.bz2
main(): Don't update commit status unless commits were pushed
There are a bunch of events that can trigger the `pull_request` webhook. These are: "assigned", "unassigned", "review_requested", "review_request_removed", "labeled", "unlabeled", "opened", "edited", "closed", or "reopened". And "synchronize". We only care about "opened" and "synchronize" because those are the only two where new commits can come through. Since we don't set commit statuses unless that commit is part of a pull request, we need a way to update the status the first time a pull request is opened, thus the "opened" action. When new commits are added we also want to update statuses, so "synchronize". But for the others, we don't want to be duplicating work and adding duplicate unnecessary statuses to PR commits.
Diffstat (limited to 'src/pull_request.rs')
0 files changed, 0 insertions, 0 deletions