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)
Thu, Apr 11, 8:46 AM
Unknown Object (File)
Thu, Apr 4, 3:39 PM
Unknown Object (File)
Thu, Mar 28, 9:39 PM
Unknown Object (File)
Jan 28 2024, 8:08 AM
Unknown Object (File)
Jan 28 2024, 8:08 AM
Unknown Object (File)
Dec 27 2023, 6:41 AM
Unknown Object (File)
Dec 21 2023, 1:51 PM
Unknown Object (File)
Dec 17 2023, 4:47 PM
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.