Page MenuHomePhabricator

Remove "arc lint --cache"
ClosedPublic

Authored by epriestley on Sep 25 2017, 6:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 12:22 AM
Unknown Object (File)
Thu, Mar 14, 4:41 PM
Unknown Object (File)
Mon, Mar 11, 2:23 AM
Unknown Object (File)
Feb 9 2024, 7:28 AM
Unknown Object (File)
Feb 6 2024, 2:00 PM
Unknown Object (File)
Feb 3 2024, 4:58 PM
Unknown Object (File)
Jan 16 2024, 7:18 AM
Unknown Object (File)
Jan 8 2024, 2:51 AM
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.