Title: [292546] trunk/Tools
- Revision
- 292546
- Author
- jbed...@apple.com
- Date
- 2022-04-07 10:37:17 -0700 (Thu, 07 Apr 2022)
Log Message
[Merge-Queue] Remove labels from pull request
https://bugs.webkit.org/show_bug.cgi?id=238909
<rdar://problem/91383048>
Reviewed by Aakash Jain.
* Tools/CISupport/ews-build/steps.py:
(RemoveLabelsFromPullRequest):
(RemoveLabelsFromPullRequest._addToLog):
(RemoveLabelsFromPullRequest.start): Remove active labels.
(RemoveLabelsFromPullRequest.getResultSummary):
(RemoveLabelsFromPullRequest.doStepIf): Only do step if pull request
number is defined.
(RemoveLabelsFromPullRequest.hideStepIf): Hide stip if step not executed.
Canonical link: https://commits.webkit.org/249384@main
Modified Paths
Diff
Modified: trunk/Tools/CISupport/ews-build/steps.py (292545 => 292546)
--- trunk/Tools/CISupport/ews-build/steps.py 2022-04-07 17:28:43 UTC (rev 292545)
+++ trunk/Tools/CISupport/ews-build/steps.py 2022-04-07 17:37:17 UTC (rev 292546)
@@ -1693,6 +1693,40 @@
return not self.doStepIf(step)
+class RemoveLabelsFromPullRequest(buildstep.BuildStep, GitHubMixin, AddToLogMixin):
+ name = 'remove-labels-from-pull-request'
+ flunkOnFailure = False
+ haltOnFailure = False
+ LABELS_TO_REMOVE = [
+ GitHubMixin.MERGE_QUEUE_LABEL,
+ GitHubMixin.FAST_MERGE_QUEUE_LABEL,
+ GitHubMixin.BLOCKED_LABEL,
+ ]
+
+ def start(self):
+ pr_number = self.getProperty('github.number', '')
+
+ repository_url = self.getProperty('repository', '')
+ rc = SUCCESS
+ if not self.remove_labels(pr_number, self.LABELS_TO_REMOVE, repository_url=repository_url):
+ rc = FAILURE
+ self.finished(rc)
+ return None
+
+ def getResultSummary(self):
+ if self.results == SUCCESS:
+ return {'step': f"Removed labels from pull request"}
+ elif self.results == FAILURE:
+ return {'step': f"Failed to remove labels from pull request"}
+ return buildstep.BuildStep.getResultSummary(self)
+
+ def doStepIf(self, step):
+ return self.getProperty('github.number') and CURRENT_HOSTNAME == EWS_BUILD_HOSTNAME
+
+ def hideStepIf(self, results, step):
+ return not self.doStepIf(step)
+
+
class CloseBug(buildstep.BuildStep, BugzillaMixin):
name = 'close-bugzilla-bug'
flunkOnFailure = False
Modified: trunk/Tools/ChangeLog (292545 => 292546)
--- trunk/Tools/ChangeLog 2022-04-07 17:28:43 UTC (rev 292545)
+++ trunk/Tools/ChangeLog 2022-04-07 17:37:17 UTC (rev 292546)
@@ -1,3 +1,20 @@
+2022-04-06 Jonathan Bedard <jbed...@apple.com>
+
+ [Merge-Queue] Remove labels from pull request
+ https://bugs.webkit.org/show_bug.cgi?id=238909
+ <rdar://problem/91383048>
+
+ Reviewed by Aakash Jain.
+
+ * CISupport/ews-build/steps.py:
+ (RemoveLabelsFromPullRequest):
+ (RemoveLabelsFromPullRequest._addToLog):
+ (RemoveLabelsFromPullRequest.start): Remove active labels.
+ (RemoveLabelsFromPullRequest.getResultSummary):
+ (RemoveLabelsFromPullRequest.doStepIf): Only do step if pull request
+ number is defined.
+ (RemoveLabelsFromPullRequest.hideStepIf): Hide stip if step not executed.
+
2022-04-07 Jonathan Bedard <jbed...@apple.com>
[Merge-Queue] Extract bug_id when updating pull-request (Follow-up fix)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
