Paths

Table of Contentst

Differential D9843

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

Authored by avivey on Jul 6 2014, 4:05 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jun 12, 12:52 PM

	Unknown Object (File)
	Mar 15 2026, 8:24 AM

	Unknown Object (File)
	Mar 14 2026, 3:37 AM

	Unknown Object (File)
	Feb 8 2026, 9:41 PM

	Unknown Object (File)
	Jan 9 2026, 4:43 PM

	Unknown Object (File)
	Jan 9 2026, 4:41 PM

	Unknown Object (File)
	Dec 28 2025, 3:32 PM

	Unknown Object (File)
	Dec 28 2025, 3:32 PM

Subscribers

Details

Reviewers

joshuaspence
epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T5124: Arcanist requires external linters to be installed even if they are not used

Commits

rARCdfdaed0b27fb: Don't build unused linters for the `ArcanistConfigurationDrivenLintEngine`

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

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1545
Build 1545: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

avivey updated this revision to Diff 23606.Jul 6 2014, 4:05 AM

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.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 6 2014, 4:05 AM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B1545: Diff 23606.Jul 6 2014, 4:06 AM

joshuaspence added a task: T5124: Arcanist requires external linters to be installed even if they are not used.Jul 6 2014, 4:33 AM

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

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.

@epriestley, are you okay with this?

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

yep

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

chad mentioned this in T6311: 'arc linters' no longer reports linters as 'Configured', all show as 'Available'.Nov 1 2014, 5:00 PM

Revision Contents
Changeset List

Path

Size

src/

lint/

engine/

ArcanistConfigurationDrivenLintEngine.php

8 lines

Diff 23606

src/lint/engine/ArcanistConfigurationDrivenLintEngine.php

Loading...