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
F13129744: D17279.id.diff
Wed, May 1, 2:18 AM
F13128771: D17279.diff
Wed, May 1, 12:20 AM
Unknown Object (File)
Mon, Apr 29, 5:36 PM
Unknown Object (File)
Sat, Apr 27, 2:38 PM
Unknown Object (File)
Tue, Apr 23, 4:02 AM
Unknown Object (File)
Fri, Apr 19, 11:17 PM
Unknown Object (File)
Fri, Apr 19, 6:11 PM
Unknown Object (File)
Mon, Apr 8, 12:11 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable