Page MenuHomePhabricator

When triggering audits, count "Accepted" revisions as successfully reviewed
ClosedPublic

Authored by epriestley on May 30 2019, 11:06 PM.
Tags
None
Referenced Files
F15494499: D20564.id49057.diff
Sun, Apr 13, 12:11 AM
F15490806: D20564.id49055.diff
Fri, Apr 11, 8:41 PM
F15474864: D20564.diff
Sun, Apr 6, 1:02 PM
F15471511: D20564.id.diff
Sat, Apr 5, 5:25 AM
F15469256: D20564.diff
Fri, Apr 4, 11:01 AM
F15439465: D20564.diff
Mar 26 2025, 7:40 AM
F15427620: D20564.id49055.diff
Mar 23 2025, 3:11 PM
F15375722: D20564.id49055.diff
Mar 12 2025, 11:49 PM
Subscribers
None

Details

Summary

See PHI1118. That issue may describe more than one bug, but the recent ordering changes to the import pipeline likely make this at least part of the problem.

Previously, commits would always close associated revisions before we made it to the "publish" step. This is no longer true, so we might be triggering audits on a commit before the associated revision actually closes.

Accommodate this by counting a revision in either "Accepted" or "Published (Was Previously Accepted)" as "reviewed".

Test Plan
  • With commit C affecting paths in package P with "Audit Unreviewed Commits and Commits With No Owner Involvement", associated with revision R, with both R and C authored by the same user, and "R" in the state "Accepted", used bin/repository reparse --publish <hash> to republish the commit.
  • Before change: audit by package P triggered.
  • After change: audit by package P no longer triggered.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable