Ref T2543. This cleans up all the "when no one is rejecting/blocking and someone accepted, mark the revision overall as accepted" logic to use more modern status stuff instead of ArcanistDifferentialRevisionStatus.
Details
Details
- Updated revisions, saw them go to "Needs Review".
- Accepted, requested changes to revisions.
- Updated one with changes requested, saw it go to "needs review" again.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- status14
- Lint
Lint Passed Severity Location Code Message Advice src/applications/differential/editor/DifferentialTransactionEditor.php:556 XHP16 TODO Comment - Unit
Tests Passed - Build Status
Buildable 18040 Build 24230: Run Core Tests Build 24229: arc lint + arc unit