Page MenuHomePhabricator

Make "Audit Requested" put commits into the "Needs Audit" state
ClosedPublic

Authored by epriestley on Jan 11 2017, 10:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 23, 4:59 AM
Unknown Object (File)
Sat, Mar 23, 4:59 AM
Unknown Object (File)
Sat, Mar 23, 4:58 AM
Unknown Object (File)
Wed, Mar 20, 11:35 PM
Unknown Object (File)
Mon, Mar 4, 6:20 PM
Unknown Object (File)
Feb 17 2024, 4:47 PM
Unknown Object (File)
Feb 15 2024, 1:31 AM
Unknown Object (File)
Feb 15 2024, 1:31 AM
Subscribers
None

Details

Summary

Fixes T7504. I think that task legitimately describes a bug and that the current behavior is counterintuitive.

Test Plan

Manually added an auditor to a commit with none; saw it become "Audit Required" as an overall state.

Diff Detail

Repository
rP Phabricator
Branch
commit10
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15213
Build 19995: Run Core Tests
Build 19994: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 11 2017, 10:58 PM
This revision was automatically updated to reflect the committed changes.