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

	F15533230: D9843.id23606.diff
	Wed, Apr 23, 6:49 PM

	F15505260: D9843.id23681.diff
	Tue, Apr 15, 1:21 AM

	F15485178: D9843.diff
	Wed, Apr 9, 10:20 PM

	F15484062: D9843.diff
	Wed, Apr 9, 3:57 PM

	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

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

Lint

Lint Skipped

Unit

Tests Skipped

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 23681

src/lint/engine/ArcanistConfigurationDrivenLintEngine.php

Loading...