Page MenuHomePhabricator

Remove "arc lint --cache"
ClosedPublic

Authored by epriestley on Sep 25 2017, 6:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 24, 1:29 AM
Unknown Object (File)
Sat, Sep 17, 6:53 PM
Unknown Object (File)
Sat, Sep 17, 2:10 PM
Unknown Object (File)
Thu, Sep 15, 5:49 PM
Unknown Object (File)
Mon, Sep 5, 8:13 AM
Unknown Object (File)
Sat, Aug 27, 5:46 PM
Unknown Object (File)
Aug 4 2022, 12:25 AM
Unknown Object (File)
Jul 31 2022, 2:02 PM
Subscribers

Details

Summary

Ref T12996. This is another half-baked Facebook-specific feature. Clear it out of the way so arc lint can be modernized more easily.

Test Plan

Grepped for cache, created this revision.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Sep 25 2017, 6:30 PM
This revision was automatically updated to reflect the committed changes.

There's still a bunch of logic in ArcanistLintEngine and ArcanistLintEngine regarding caching... should that be removed/deprecated as well?

Yeah. I think granularity on lint messages gets a mention in T13098, at least. I don't currently plan to rebuild support for this stuff, at least until all the dust has settled on the whole huge tree of issues under T13098.