Paths

Table of Contentst

Differential D14220

Fix ArcanistClosureLinter
ClosedPublic
Actions

Authored by vm on Oct 2 2015, 3:30 AM.

Tags

None

Referenced Files

	F19711887: D14220.diff
	Wed, Feb 11, 7:41 PM

	F19672238: D14220.diff
	Sun, Feb 8, 2:54 AM

	F19582184: D14220.diff
	Sun, Feb 1, 9:28 PM

	F19580771: D14220.id34343.diff
	Sun, Feb 1, 5:51 PM

	F19554790: D14220.id34343.diff
	Jan 30 2026, 3:12 AM

	F19554789: D14220.id.diff
	Jan 30 2026, 3:12 AM

	F19554779: D14220.diff
	Jan 30 2026, 3:05 AM

	F19543250: D14220.diff
	Jan 22 2026, 5:25 PM

Subscribers

Details

Reviewers

joshuaspence
epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T9498: ArcanistClosureLinter broken after D14165

Commits

rARCf0c57986bc32: Fix ArcanistClosureLinter

Summary

Fixes T9498

Test Plan

Run arc lint with errors that get caught and reported by ArcanistClosureLinter

Diff Detail

Repository

Branch

fix_closure_linter

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 8146
Build 9302: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	159 ms	ArcanistCppcheckLinterTestCase::Unknown Unit Message ("")
	138 ms	ArcanistCpplintLinterTestCase::Unknown Unit Message ("")
	891 ms	ArcanistJSHintLinterTestCase::Unknown Unit Message ("")
	661 ms	ArcanistJSONLintLinterTestCase::Unknown Unit Message ("")
	146 ms	ArcanistPyFlakesLinterTestCase::Unknown Unit Message ("")
		View Full Test Results (5 Failed · 31 Passed · 19 Skipped)

Event Timeline

vm updated this revision to Diff 34338.Oct 2 2015, 3:30 AM

vm retitled this revision from to Fix ArcanistClosureLinter.

vm updated this object.

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

vm added a reviewer: epriestley.

vm added a task: T9498: ArcanistClosureLinter broken after D14165.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptOct 2 2015, 3:30 AM

Herald added a subscriber: Korvin. · View Herald Transcript

joshuaspence added a reviewer: joshuaspence.Oct 2 2015, 8:31 AM

epriestley accepted this revision.Oct 2 2015, 12:45 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Oct 2 2015, 12:45 PM

Closed by commit rARCf0c57986bc32: Fix ArcanistClosureLinter (authored by vm, committed by epriestley). · Explain WhyOct 2 2015, 12:45 PM

This revision was automatically updated to reflect the committed changes.

vm mentioned this in D14225: Set name on more linters.Oct 2 2015, 3:57 PM

epriestley mentioned this in rARC1b4a3e0c5eb9: Set path on more linters.Oct 2 2015, 3:58 PM

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistClosureLinter.php

1 line

Diff 34338

src/lint/linter/ArcanistClosureLinter.php

Loading...