Page MenuHomePhabricator
Differential D12308

Further improvements to array comma linter rule
ClosedPublic
Actions

Authored by joshuaspence on Apr 6 2015, 9:56 PM.
Tags
None
Referenced Files
F18737834: D12308.id31149.diff
Wed, Oct 1, 12:54 PM
F18717371: D12308.id29565.diff
Mon, Sep 29, 1:51 PM
F18647591: D12308.id31149.diff
Fri, Sep 19, 1:44 PM
F18631246: D12308.diff
Tue, Sep 16, 2:49 PM
F18624253: D12308.id.diff
Mon, Sep 15, 7:24 PM
F18621607: D12308.id.diff
Mon, Sep 15, 8:56 AM
F18621325: D12308.id.diff
Mon, Sep 15, 7:47 AM
F18611189: D12308.diff
Sun, Sep 14, 6:21 AM
View All Files
Subscribers
epriestley

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC993166fa497e: Further improvements to array comma linter rule
Summary

Improve the ArcanistXHPASTLinter::LINT_ARRAY_SEPARATOR rule to be able to autofix the following code:

array(
  1,
  2,
  3, /* comment */ );
Test Plan

Added unit tests.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 29565.Apr 6 2015, 9:56 PM
joshuaspence retitled this revision from to Further improvements to array comma linter rule.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 6 2015, 9:56 PM
Herald added a subscriber: epriestley. · View Herald Transcript
• nagym718 added a task: T5042: Futures()->limit(X) does not ensure that X number of tasks are running after the first set of tasks.Apr 7 2015, 8:03 AM
epriestley removed a task: T5042: Futures()->limit(X) does not ensure that X number of tasks are running after the first set of tasks.Apr 7 2015, 12:25 PM
joshuaspence updated this revision to Diff 30962.May 16 2015, 1:34 AM
joshuaspence edited edge metadata.

Works now

joshuaspence updated this object.May 19 2015, 1:24 PM
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley accepted this revision.May 19 2015, 3:14 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 19 2015, 3:14 PM
Closed by commit rARC993166fa497e: Further improvements to array comma linter rule (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 19 2015, 9:05 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
2 lines
src/
lint/
linter/
48 lines
__tests__/
xhpast/
10 lines

Diff 31149

View Options

.editorconfig

Loading...
View Options

src/lint/linter/ArcanistXHPASTLinter.php

Loading...
View Options

src/lint/linter/__tests__/xhpast/array-comma.lint-test

Loading...
Phabricator · Built by Phacility