<div dir="ltr">Good morning reviewers,<div><br></div><div style>Please be sure always to look at the Previous Ticket report for a given ticket, and check the status of the previous review. Two reasons:</div><div style><br>
</div><div style>One, we allow the flexibility for a reviewer to use subjective judgement to close a ticket as "approved" if only minor "required" issues are found, on the condition that the developer address those "required" issues in the next revision. Thus, it is imperative that such allowances not fall through the cracks.</div>
<div style><br></div><div style>This is especially important for previously approved (Priority #1) tickets, since such tickets already receive an expedited, diff-only review.</div><div style><br></div><div style>Two, we have a great system in place, that allows a reviewer to hold a ticket open if only minor issues need to be addressed, such that once a revision is submitted, the reviewer can continue the current review in the new ticket. If the previous ticket is still open, the new ticket should be left for the reviewer of the previous ticket, so that the previous review may be continued.</div>
<div style><br></div><div style>I had two such tickets over the weekend: both had previous-ticket "required" issues that were not addressed. I left the tickets open to allow for a review continuation. Both developers uploaded revisions, but when I went in this morning to take the tickets, both had been assigned, reviewed, and closed. Unfortunately, in both cases, the review failed to indicate whether previous-ticket "required" issues had been addressed.</div>
<div style><br></div><div style>Thanks, as always, for all of your contributions!</div><div style><br></div><div style>Chip</div></div>