HomePhabricator

Add a linter rule for array separators

Description

Add a linter rule for array separators

Summary:
Adds a rule to ArcanistXHPASTLinter which ensures that:

  1. Single-lined arrays do not contain an unnecessary trailing comma separator after the final array value.
  2. Multi-lined arrays do contain a trailing comma seperator after the final value value.

Depends on D10534.

Test Plan: Wrote and executed unit tests.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: epriestley, Korvin

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

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Sep 24 2014, 9:41 PM
Reviewer
Blessed Reviewers
Differential Revision
D10535: Add a linter rule for array separators
Parents
rARC13492c8043a4: Fix a fatal error in `ArcanistXHPASTLinter`
Branches
Unknown
Tags
Unknown

Event Timeline