Differential D11504

Add a linter rule for PHP indentation
AbandonedPublic
Actions

Authored by joshuaspence on Jan 26 2015, 10:12 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jan 1, 6:20 PM

	Unknown Object (File)
	Tue, Dec 31, 4:41 AM

	Unknown Object (File)
	Sat, Dec 28, 3:06 PM

	Unknown Object (File)
	Sun, Dec 8, 4:23 PM

	Unknown Object (File)
	Fri, Dec 6, 3:25 AM

	Unknown Object (File)
	Thu, Dec 5, 1:07 PM

	Unknown Object (File)
	Dec 2 2024, 3:33 PM

	Unknown Object (File)
	Nov 30 2024, 12:24 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Summary

WIP. Depends on D12295.

Test Plan

WIP

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 4288
Build 4301: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed
View All

	Time	Test
	5,849 ms	ArcanistXHPASTLinterTestCase::testLinter
	720 ms	ArcanistCSSLintLinterTestCase::testLinter
	192 ms	ArcanistCSSLintLinterTestCase::testVersion
	161 ms	ArcanistChmodLinterTestCase::testLinter
	275 ms	ArcanistClosureLinterTestCase::testLinter
		View Full Test Results (1 Failed · 44 Passed · 6 Skipped)

Event Timeline

joshuaspence updated this revision to Diff 27664.Jan 26 2015, 10:12 AM

joshuaspence retitled this revision from to Add a linter rule for PHP indentation.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 26 2015, 10:12 AM

Herald added a subscriber: epriestley. · View Herald Transcript

joshuaspence mentioned this in D10440: Add an XHPAST linter rule to ensure that only one statement exists per line.Jan 26 2015, 10:13 AM

Just pushing this out of my queue for now, I'm excited about it if you can make it work.

I'm wildly amused by this diff getting flagged by itself.

This revision now requires changes to proceed.Jan 26 2015, 3:14 PM

Somewhat working

Herald added a subscriber: Korvin. · View Herald TranscriptJan 28 2015, 11:15 AM

Somewhat working

Further work, still WIP

joshuaspence mentioned this in D12295: Add a method to get the indentation for an XHPASTNode.Apr 5 2015, 12:15 PM

joshuaspence mentioned this in rPHUd30d314dc8eb: Add a method to get the indentation for an XHPASTNode.Apr 6 2015, 9:44 PM

Rebase, still not working

joshuaspence updated this object.Jun 1 2015, 9:08 PM

joshuaspence added a parent revision: D12295: Add a method to get the indentation for an XHPASTNode.

Further work, seems to work reasonably well but still needs work

Not actively working on this.

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistXHPASTLinter.php

121 lines

__tests__/

xhpast/

call-time-pass-by-reference.lint-test

6 lines

duplicate-switch-case.lint-test

40 lines

indentation.lint-test

50 lines

php54-features.lint-test

2 lines

switches.lint-test

18 lines

undeclared-variables.lint-test

16 lines

Diff 28041

src/lint/linter/ArcanistXHPASTLinter.php

Loading...

src/lint/linter/tests/xhpast/call-time-pass-by-reference.lint-test

Loading...

src/lint/linter/tests/xhpast/duplicate-switch-case.lint-test

Loading...

src/lint/linter/tests/xhpast/indentation.lint-test

Loading...

src/lint/linter/tests/xhpast/php54-features.lint-test

Loading...

src/lint/linter/tests/xhpast/switches.lint-test

Loading...

src/lint/linter/tests/xhpast/undeclared-variables.lint-test

Loading...