Page MenuHomePhabricator

When an Owners package accepts a revision, count that as an "involved owner" for the purposes of audit
ClosedPublic

Authored by epriestley on Feb 7 2019, 9:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 9:04 PM
Unknown Object (File)
Wed, Dec 11, 7:01 PM
Unknown Object (File)
Mon, Dec 2, 10:24 AM
Unknown Object (File)
Wed, Nov 27, 1:17 PM
Unknown Object (File)
Nov 19 2024, 1:07 AM
Unknown Object (File)
Oct 21 2024, 12:49 PM
Unknown Object (File)
Oct 20 2024, 1:23 PM
Unknown Object (File)
Oct 17 2024, 11:28 AM
Subscribers
None

Details

Summary

Depends on D20129. Ref T13244. See PHI1058. When a revision has an "Accept" from a package, count the owners as "involved" in the change whether or not any actual human owners are actually accepting reviewers.

If a user owns "/" and uses "force accept" to cause "/src/javascript" to accept, or a user who legitimately owns "/src/javascript" accepts on behalf of the package but not on behalf of themselves (for whatever reason), it generally makes practical sense that these changes have owners involved in them (i.e., that's what a normal user would expect in both cases) and don't need to trigger audits under "no involvement" rules.

Test Plan

Used bin/repository reparse --force --owners <commit> to trigger audit logic. Saw a commit owned by O1 with a revision counted as "involved" when O1 had accepted the revision, even though no actual human owner had accepted it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable