Page MenuHomePhabricator

Excluded authored commits from "Ready to Audit"; handle unreachable commits better
ClosedPublic

Authored by epriestley on Jan 31 2017, 9:19 PM.
Tags
None
Referenced Files
F14758057: D17279.diff
Wed, Jan 22, 4:48 AM
F14750344: D17279.id.diff
Tue, Jan 21, 11:47 AM
F14745048: D17279.diff
Tue, Jan 21, 9:17 AM
Unknown Object (File)
Thu, Jan 16, 8:19 PM
Unknown Object (File)
Thu, Jan 16, 8:18 PM
Unknown Object (File)
Thu, Jan 16, 7:20 PM
Unknown Object (File)
Nov 19 2024, 3:22 PM
Unknown Object (File)
Oct 19 2024, 7:54 PM
Subscribers
None

Details

Summary

Ref T10978. I'm inching toward cleaning up our audit state. Two issues are:

  • Authored commits show up in "Ready to Audit", but should not.
  • Unreachable commits (like that stacked of unsquashed stuff) show up too, but we don't really care about them.

Kick authored stuff out of the "Ready to Audit" bucket and hide unreachable commits by default, with constraints for filtering. Also give them a closed/disabled/strikethru style.

Test Plan
  • Viewed audit buckets.
  • Searched for reachable/unreachable commits.

Diff Detail

Repository
rP Phabricator
Branch
audit16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15465
Build 20380: Run Core Tests
Build 20379: arc lint + arc unit