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
F15470646: D13868.diff
Sat, Apr 5, 12:05 AM
F15464914: D13868.diff
Wed, Apr 2, 1:04 PM
F15455492: D13868.id33873.diff
Sun, Mar 30, 1:04 AM
F15450599: D13868.id33495.diff
Fri, Mar 28, 4:33 PM
F15450501: D13868.id.diff
Fri, Mar 28, 4:03 PM
F15447044: D13868.diff
Thu, Mar 27, 9:33 PM
F15390442: D13868.id33495.diff
Mar 15 2025, 6:13 AM
Unknown Object (File)
Feb 19 2025, 12:39 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.