Page MenuHomePhabricator
Differential D14220

Fix ArcanistClosureLinter
ClosedPublic
Actions

Authored by vm on Oct 2 2015, 3:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 1:39 AM
Unknown Object (File)
Wed, Apr 24, 9:56 PM
Unknown Object (File)
Sun, Apr 21, 3:53 PM
Unknown Object (File)
Fri, Apr 19, 11:33 PM
Unknown Object (File)
Fri, Apr 19, 3:42 PM
Unknown Object (File)
Wed, Apr 17, 2:56 PM
Unknown Object (File)
Thu, Apr 11, 7:09 AM
Unknown Object (File)
Apr 4 2024, 3:08 AM
View All Files
Subscribers
Korvin

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
rARC Arcanist
Branch
fix_closure_linter
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 8146
Build 9302: arc lint + arc unit

Unit TestsFailed
View All

TimeTest
159 msArcanistCppcheckLinterTestCase::Unknown Unit Message ("")
138 msArcanistCpplintLinterTestCase::Unknown Unit Message ("")
891 msArcanistJSHintLinterTestCase::Unknown Unit Message ("")
661 msArcanistJSONLintLinterTestCase::Unknown Unit Message ("")
146 msArcanistPyFlakesLinterTestCase::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

PathSize
src/
lint/
linter/
1 line

Diff 34338

View Options

src/lint/linter/ArcanistClosureLinter.php

Loading...
Phabricator · Built by Phacility