[EWS] Ignore hooks from closed PRs
https://bugs.webkit.org/show_bug.cgi?id=235666
<rdar://problem/88091611>
Reviewed by Aakash Jain.
* Tools/CISupport/ews-build/events.py:
(GitHubEventHandlerNoEdits):
(GitHubEventHandlerNoEdits.handle_pull_request): Ignore hooks coming from closed pull requests.
Canonical link: https://commits.webkit.org/246465@main
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@288657 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/Tools/CISupport/ews-build/events.py b/Tools/CISupport/ews-build/events.py
index b92fbb2..3bf4248 100644
--- a/Tools/CISupport/ews-build/events.py
+++ b/Tools/CISupport/ews-build/events.py
@@ -313,6 +313,7 @@
class GitHubEventHandlerNoEdits(GitHubEventHandler):
ACTIONS_TO_TRIGGER_EWS = ('opened', 'synchronize')
+ OPEN_STATES = ('open',)
def _get_commit_msg(self, repo, sha):
return ''
@@ -320,7 +321,11 @@
def handle_pull_request(self, payload, event):
pr_number = payload['number']
action = payload.get('action')
+ state = payload.get('state')
if action not in self.ACTIONS_TO_TRIGGER_EWS:
log.msg('Action {} on PR #{} does not indicate code has been changed'.format(action, pr_number))
return ([], 'git')
+ if state not in self.OPEN_STATES:
+ log.msg("PR #{} is '{}', which triggers nothing".format(pr_number, state))
+ return ([], 'git')
return super(GitHubEventHandlerNoEdits, self).handle_pull_request(payload, event)
diff --git a/Tools/ChangeLog b/Tools/ChangeLog
index ed7dc7d..44f2d9d 100644
--- a/Tools/ChangeLog
+++ b/Tools/ChangeLog
@@ -1,3 +1,15 @@
+2022-01-26 Jonathan Bedard <jbedard@apple.com>
+
+ [EWS] Ignore hooks from closed PRs
+ https://bugs.webkit.org/show_bug.cgi?id=235666
+ <rdar://problem/88091611>
+
+ Reviewed by Aakash Jain.
+
+ * CISupport/ews-build/events.py:
+ (GitHubEventHandlerNoEdits):
+ (GitHubEventHandlerNoEdits.handle_pull_request): Ignore hooks coming from closed pull requests.
+
2022-01-26 Don Olmstead <don.olmstead@sony.com>
Share TestRunner::installFakeHelvetica implementation