Differential D15299

Disable linter rules by default if a linter standard is defined
AcceptedPublic
Actions

Authored by joshuaspence on Feb 18 2016, 4:37 AM.

Tags

None

Referenced Files

	F18768602: D15299.id.diff
	Oct 8 2025, 4:26 AM

	F18761220: D15299.diff
	Oct 6 2025, 12:59 PM

	F18726582: D15299.id36906.diff
	Sep 30 2025, 6:44 AM

	F18626986: D15299.diff
	Sep 16 2025, 2:55 AM

	F18516978: D15299.diff
	Sep 5 2025, 1:06 PM

	F18500024: D15299.diff
	Sep 4 2025, 8:17 PM

	F18426529: D15299.id36906.diff
	Aug 30 2025, 11:00 PM

	F18424801: D15299.diff
	Aug 30 2025, 9:22 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Summary

If a linter standard is defined, disable all linter rules by default. This ensures that only rules whitelisted by the linter standard are used.

Test Plan

N/A

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 10749
Build 13226: Run Core Tests
Build 13225: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	1,578 ms	ArcanistPuppetLintLinterTestCase::Unknown Unit Message ("")
	24 ms	ArcanistAbstractMethodBodyXHPASTLinterRuleTestCase::Unknown Unit Message ("")
	10 ms	ArcanistAbstractPrivateMethodXHPASTLinterRuleTestCase::Unknown Unit Message ("")
	10 ms	ArcanistAliasFunctionXHPASTLinterRuleTestCase::Unknown Unit Message ("")
	10 ms	ArcanistArrayCombineXHPASTLinterRuleTestCase::Unknown Unit Message ("")
		View Full Test Results (1 Failed · 209 Passed · 44 Skipped)

Event Timeline

joshuaspence updated this revision to Diff 36906.Feb 18 2016, 4:37 AM

joshuaspence retitled this revision from to Disable linter rules by default if a linter standard is defined.

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptFeb 18 2016, 4:37 AM

Herald added a subscriber: Korvin. · View Herald Transcript

epriestley accepted this revision.Feb 18 2016, 10:56 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Feb 18 2016, 10:56 AM

• Persona added a child revision: D15341: Fix Ponder Exception, spacing.Feb 24 2016, 11:16 AM

tycho.tatitscheff removed a child revision: D15341: Fix Ponder Exception, spacing.Feb 24 2016, 12:31 PM

• TweikerManager added a commit: rP2604c5af55f6: (stable) Promote 2017 Week 4.Jan 29 2017, 10:50 PM

epriestley removed a commit: rP2604c5af55f6: (stable) Promote 2017 Week 4.Jan 29 2017, 10:52 PM

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

src/

lint/

linter/

ArcanistLinter.php

7 lines

46%

Loading...

Diff 36906

src/lint/linter/ArcanistLinter.php

Loading...