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
Unknown Object (File)
Wed, Jul 24, 2:03 PM
Unknown Object (File)
Tue, Jul 23, 12:17 AM
Unknown Object (File)
Mon, Jul 22, 12:44 PM
Unknown Object (File)
Mon, Jul 22, 10:54 AM
Unknown Object (File)
Sun, Jul 21, 1:56 AM
Unknown Object (File)
Sat, Jul 20, 7:34 AM
Unknown Object (File)
Sat, Jul 20, 4:23 AM
Unknown Object (File)
Sat, Jul 20, 3:26 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