Page MenuHomePhabricator

Don't build unused linters for the `ArcanistConfigurationDrivenLintEngine`
ClosedPublic

Authored by avivey on Jul 6 2014, 4:05 AM.
Tags
None
Referenced Files
F15454049: D9843.id23606.diff
Sat, Mar 29, 3:59 PM
F15451795: D9843.id.diff
Fri, Mar 28, 11:51 PM
F15449876: D9843.id23681.diff
Fri, Mar 28, 12:37 PM
F15443623: D9843.diff
Thu, Mar 27, 5:18 AM
F15407747: D9843.id23606.diff
Tue, Mar 18, 7:20 PM
F15383704: D9843.diff
Fri, Mar 14, 5:38 PM
F15382798: D9843.id23606.diff
Fri, Mar 14, 2:31 PM
Unknown Object (File)
Feb 24 2025, 9:12 AM
Subscribers

Details

Summary

Fixes T5124.

Test Plan

arc lint on a Phabricator diff that doesn't have JS files doesn't complain on missing JSHint.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

avivey retitled this revision from to Configuration Driven: Don't build unused linters.
avivey updated this object.
avivey edited the test plan for this revision. (Show Details)
avivey added reviewers: epriestley, joshuaspence.
joshuaspence edited edge metadata.

This looks good to me.

avivey retitled this revision from Configuration Driven: Don't build unused linters to Configuration Driven Lint: Don't build unused linters.Jul 6 2014, 7:48 AM
avivey edited edge metadata.
joshuaspence retitled this revision from Configuration Driven Lint: Don't build unused linters to Don't build unused linters for the `ArcanistConfigurationDrivenLintEngine`.Jul 9 2014, 4:43 PM
epriestley updated this revision to Diff 23681.

Closed by commit rARCdfdaed0b27fb (authored by @avivey, committed by @epriestley).