-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ews-build.webkit.org] Convert SetCommitQueueMinusFlagOnPatch to Twisted asynchronous requests #9317
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r+ assuming this is tested
def set_cq_minus_flag_on_patch(self, patch_id): | ||
patch_url = '{}rest/bug/attachment/{}'.format(BUG_SERVER_URL, patch_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might be good to keep the URL is explicit variable (like before) for maintainability.
EWS run on previous version of this PR (hash f7ecac8)
|
f7ecac8
to
709a90c
Compare
EWS run on previous version of this PR (hash 709a90c)
|
709a90c
to
80af614
Compare
EWS run on current version of this PR (hash 80af614)
|
…ted asynchronous requests https://bugs.webkit.org/show_bug.cgi?id=251294 rdar://104761643 Reviewed by Aakash Jain. * Tools/CISupport/ews-build/steps.py: (BugzillaMixin.set_cq_minus_flag_on_patch): Convert to Twisted asynchronous requests. (SetCommitQueueMinusFlagOnPatch.run): Renamed from start. (SetCommitQueueMinusFlagOnPatch.start): Renamed to run. Canonical link: https://commits.webkit.org/259729@main
80af614
to
3465693
Compare
Committed 259729@main (3465693): https://commits.webkit.org/259729@main Reviewed commits have been landed. Closing PR #9317 and removing active labels. |
3465693
80af614
🧪 style🛠 ios🛠 mac🛠 wpe🛠 🧪 win🧪 bindings🛠 ios-sim🛠 mac-AS-debug🛠 gtk🛠 wincairo🧪 webkitperl🧪 ios-wk2🧪 api-mac🧪 gtk-wk2🧪 api-ios🧪 mac-wk1🧪 api-gtk🛠 tv🧪 mac-wk2🛠 tv-sim🧪 mac-AS-debug-wk2🛠 watch🧪 mac-wk2-stress🛠 watch-sim