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
F15487434: D19646.id46952.diff
Thu, Apr 10, 1:58 PM
F15450987: D19646.id46952.diff
Fri, Mar 28, 7:29 PM
F15437982: D19646.id.diff
Mar 25 2025, 10:28 PM
F15433356: D19646.diff
Mar 24 2025, 10:49 PM
F15405057: D19646.diff
Mar 18 2025, 10:08 AM
F15378857: D19646.id46957.diff
Mar 13 2025, 5:22 PM
F15352865: D19646.id46957.diff
Mar 10 2025, 6:51 PM
F15333260: D19646.diff
Mar 8 2025, 1:23 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