Bug 235377
| Summary: | [EWS] ConfigureBuild should display PR information instead of ValidateChange | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> |
| Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | aakash_jain, jbedard, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=234847 https://bugs.webkit.org/show_bug.cgi?id=235578 |
||
Aakash Jain
ConfigureBuild and ValidateChange steps both display same information, about the PR. For e.g.: in https://ews-build.webkit-uat.org/#/builders/1/builds/1687
ConfigureBuild display: "Pull request 78", and ValidateChange display: "[PR 78] [webkitbugspy] Support automted comments" Both link to same url, and duplicate the information displayed. Either one of the step should display this info.
Ideally ConfigureBuild should display all these information if it can. In bugzilla work flow ConfigureBuild didn't had bug title information, that's why it couldn't display it and ValidatePatch display it. But in git workflow ConfigureBuild would probably have the required information. So we can display all these information together in ConfigureBuild step and not display such info in ValidateChange step. This would be more valuable in the builds in which RelevanceCheck fails.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/87853245>
Jonathan Bedard
Pull-request: https://github.com/WebKit/WebKit/pull/84
Jonathan Bedard
Landed 246269@main (r288367)