Page MenuHomePhabricator
Differential D18643

Remove "arc lint --cache"
ClosedPublic
Actions

Authored by epriestley on Sep 25 2017, 6:22 PM.
Tags
None
Referenced Files
F18342090: D18643.diff
Tue, Aug 26, 3:27 AM
F18340654: D18643.id44776.diff
Mon, Aug 25, 10:22 PM
F18234261: D18643.id44776.diff
Wed, Aug 20, 5:46 PM
F18094914: D18643.id44758.diff
Thu, Aug 7, 10:25 PM
Unknown Object (File)
Jun 17 2025, 11:32 AM
Unknown Object (File)
Jun 17 2025, 3:28 AM
Unknown Object (File)
Apr 26 2025, 11:21 AM
Unknown Object (File)
Apr 23 2025, 11:17 PM
View All Files
Subscribers
joshuaspence

Details

Reviewers
amckinley
Maniphest Tasks
T12996: Parallelize substages of `arc diff`
Commits
rARCbe7987b25a82: Remove "arc lint --cache"
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

epriestley created this revision.Sep 25 2017, 6:22 PM
Harbormaster completed remote builds in B18536: Diff 44758.Sep 25 2017, 6:22 PM
amckinley accepted this revision.Sep 25 2017, 6:30 PM
This revision is now accepted and ready to land.Sep 25 2017, 6:30 PM
Closed by commit rARCbe7987b25a82: Remove "arc lint --cache" (authored by epriestley). · Explain WhySep 27 2017, 3:21 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in D18651: Make "arc diff" sort of detect dependent revisions.Sep 27 2017, 5:05 PM
epriestley mentioned this in rARC5ddc5732239a: Make "arc diff" sort of detect dependent revisions.Sep 27 2017, 5:46 PM
joshuaspence added a subscriber: joshuaspence.Apr 8 2018, 12:48 AM

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

epriestley added a comment.Apr 8 2018, 12:50 AM

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.

Revision Contents
Changeset List

PathSize
src/
configuration/
13 lines
workflow/
100 lines

Diff 44776

View Options

src/configuration/ArcanistSettings.php

Loading...
View Options

src/workflow/ArcanistLintWorkflow.php

Loading...
Phabricator · Built by Phacility