Bug 245677

Summary: Reviewers who approve a PR after `validate-commiter-and-reviewer` should be included in the final Reviewed by line
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: jbedard, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 239082    

Description Wenson Hsieh 2022-09-26 09:07:51 PDT
In the GitHub workflow, if you have a patch that's

- Reviewed by A
- Tagged with `merge-queue`
- Passes the `validate-commiter-and-reviewer` check in merge queue.
- Reviewed by B
- Lands

...the "Reviewed by" line will only say "Reviewed by A", but it would be ideal to have "Reviewed by A and B".
Comment 1 Radar WebKit Bug Importer 2022-10-03 09:08:21 PDT
<rdar://problem/100719648>
Comment 2 Jonathan Bedard 2023-01-18 07:53:34 PST
I guess this would entail running the `validate-committer-and-reviewer` check right before landing.