HomePhabricator

Minor improvement for array value linter rule

Description

Minor improvement for array value linter rule

Summary:
This linter rule fails on multi-line arrays with no whitespace before the first array value. Specifically, the following exception is thrown:

Fatal error: Call to a member function isAnyWhitespace() on boolean in arcanist/src/lint/linter/xhpast/rules/ArcanistArrayValueXHPASTLinterRule.php on line 40

Test Plan: Added another test case.

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D13856

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Aug 10 2015, 9:55 PM
Reviewer
Blessed Reviewers
Differential Revision
D13856: Minor improvement for array value linter rule
Parents
rARC830bcbc2a5e9: Add a linter rule for array elements
Branches
Unknown
Tags
Unknown