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
F17312017: D13868.diff
Thu, Jun 26, 1:16 PM
Unknown Object (File)
Fri, May 30, 2:39 PM
Unknown Object (File)
Fri, May 30, 2:17 PM
Unknown Object (File)
Thu, May 29, 7:19 PM
Unknown Object (File)
May 6 2025, 9:20 PM
Unknown Object (File)
May 1 2025, 7:52 AM
Unknown Object (File)
Apr 28 2025, 3:09 PM
Unknown Object (File)
Apr 28 2025, 2:22 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.