247900 2022-11-14 08:11:28 -0800 Add support for skip-ews GitHub label 2023-01-26 11:26:28 -0800 1 1 1 Unclassified WebKit Tools / Tests Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=250859 InRadar P2 Normal --- 1 aakash_jain jbedard aakash_jain ap jbedard mark.lam ryanhaddad webkit-bug-importer oldest_to_newest 1912316 0 aakash_jain 2022-11-14 08:11:28 -0800 Add support for Skip-EWS GitHub label. This label can be applied to a PR to skip EWS runs. There might be various scenarios in which people might want to apply such a label. Foe e.g: - Some engineers might want to abort EWS as soon as the they know that the change is going to be bad (e.g: an important ews on their patch failed, and they don't want to keep running on all other ewses). - Someone just re-uploaded the change after doing a trivial change (add reviewer name, fix typo in PR title etc.) - Admins wants to stop a bad PR (which might be breaking EWS bots) from further processing 1912318 1 aakash_jain 2022-11-14 08:14:05 -0800 Note that, in bugzilla world, there are three ways to do similar thing: 1) r- the patch 2) mark the patch as obsolete from bugzilla UI 3) --no-ews while uploading the patch In GitHub world, we don't have any way to do this. Equivalent of r- in GitHub is marking the PR as Request changes, which still triggers the EWS. There is no way to mark a commit as obsolete in GitHub UI without uploading a new commit. 1912321 2 aakash_jain 2022-11-14 08:19:44 -0800 Pull request: https://github.com/WebKit/WebKit/pull/6476 1913002 3 ews-feeder 2022-11-16 17:26:13 -0800 Committed 256760@main (57ba798e7aa4): <https://commits.webkit.org/256760@main> Reviewed commits have been landed. Closing PR #6476 and removing active labels. 1913003 4 webkit-bug-importer 2022-11-16 17:27:17 -0800 <rdar://problem/102444473> 1913332 5 jbedard 2022-11-17 16:25:11 -0800 Re-opening for pull request https://github.com/apple/WebKit/pull/216 1913463 6 ews-feeder 2022-11-18 08:37:23 -0800 Committed 252432.837@safari-7614-branch (8dbf19696f48): <https://commits.webkit.org/252432.837@safari-7614-branch> Reviewed commits have been landed. Closing PR #216 and removing active labels.