Page MenuHomePhabricator

Introduce an AuditStatus object for commits and move some callsites to it
ClosedPublic

Authored by epriestley on Sep 7 2018, 3:37 PM.
Tags
None
Referenced Files
F18999542: D19646.diff
Thu, Nov 20, 9:19 PM
F18993068: D19646.id.diff
Wed, Nov 19, 5:29 AM
F18832447: D19646.id46957.diff
Oct 25 2025, 9:55 PM
F18832446: D19646.id46952.diff
Oct 25 2025, 9:54 PM
F18832445: D19646.id.diff
Oct 25 2025, 9:54 PM
F18831617: D19646.diff
Oct 25 2025, 1:49 PM
F18702459: D19646.id46957.diff
Sep 28 2025, 12:14 AM
F18600494: D19646.diff
Sep 13 2025, 9:52 AM
Subscribers
None

Details

Summary

Ref T13195. See PHI851. Add an object, analogous to the DifferentialRevisionStatus object, to handle audit status management.

This will primarily make it easier to swap storage over to strings later, but also cleans things up a bit.

Test Plan

Viewed audit/commit lists, saw sensible state icons. Ran bin/audit synchronize, got sensible output.

Diff Detail

Repository
rP Phabricator
Branch
audit4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20760
Build 28228: Run Core Tests
Build 28227: arc lint + arc unit