Page MenuHomePhabricator

Remove leftover code for "postponed" lint and unit status
ClosedPublic

Authored by joshuaspence on Aug 11 2015, 1:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 3, 10:38 PM
Unknown Object (File)
Wed, Jan 1, 2:28 AM
Unknown Object (File)
Fri, Dec 20, 8:11 PM
Unknown Object (File)
Fri, Dec 13, 6:40 AM
Unknown Object (File)
Nov 26 2024, 4:27 PM
Unknown Object (File)
Nov 12 2024, 7:20 AM
Unknown Object (File)
Nov 11 2024, 6:58 AM
Unknown Object (File)
Nov 11 2024, 5:58 AM
Subscribers

Details

Summary

Ref T9134. It looks like this functionality was removed in D13848. Depends on D13869.

Test Plan

Ran arc diff, arc lint and arc unit.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Remove leftover code for "postponed" lint and unit status.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

I believe (although I haven't verified) that this is a "more breaking" change than what's in Phabricator HEAD (which may fail server-side integrations and remove information from the UI, but won't generally affect "arc"), so maybe hold this for a bit. But whatever. I believe the number of installs affected by this is very small (and maybe even 1 or 0) and we've never documented or recommended this mechanism since it has been on the chopping block for years.

This revision is now accepted and ready to land.Aug 11 2015, 1:56 PM
This revision was automatically updated to reflect the committed changes.