Paths

Table of Contentst

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

	F15557592: D12308.id31149.diff
	Mon, Apr 28, 10:19 PM

	F15556691: D12308.id30962.diff
	Mon, Apr 28, 5:57 PM

	F15550630: D12308.id30962.diff
	Sun, Apr 27, 12:48 PM

	F15549279: D12308.id29565.diff
	Sun, Apr 27, 4:50 AM

	F15545124: D12308.id.diff
	Sat, Apr 26, 9:02 AM

	F15540368: D12308.diff
	Fri, Apr 25, 7:15 AM

	F15512270: D12308.id31149.diff
	Thu, Apr 17, 9:22 AM

	F15506114: D12308.id31149.diff
	Tue, Apr 15, 8:45 AM

Subscribers

Details

Reviewers

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

Branch

master

Lint

No Lint Coverage

Unit

Test Failures

Build Status

Buildable 5167
Build 5185: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed
View All

	Time	Test
	8,860 ms	ArcanistXHPASTLinterTestCase::testLinter
	816 ms	ArcanistCSSLintLinterTestCase::testLinter
	119 ms	ArcanistCSSLintLinterTestCase::testVersion
	126 ms	ArcanistChmodLinterTestCase::testLinter
	217 ms	ArcanistClosureLinterTestCase::testLinter
		View Full Test Results (1 Failed · 46 Passed · 6 Skipped)

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

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

Path

Size

src/

lint/

linter/

__tests__/

xhpast/

array-comma.lint-test

10 lines

Diff 29565

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

Loading...