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

	F14047756: D11504.id27759.diff
	Thu, Nov 14, 5:05 AM

	F14047697: D11504.id27664.diff
	Thu, Nov 14, 4:57 AM

	F14047696: D11504.id31635.diff
	Thu, Nov 14, 4:56 AM

	F14047693: D11504.id28041.diff
	Thu, Nov 14, 4:56 AM

	F14047692: D11504.id30205.diff
	Thu, Nov 14, 4:56 AM

	F14047319: D11504.id.diff
	Thu, Nov 14, 3:30 AM

	F14047282: D11504.id27758.diff
	Thu, Nov 14, 3:19 AM

	F14047269: D11504.diff
	Thu, Nov 14, 3:14 AM

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 4114
Build 4127: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed
View All

	Time	Test
	1,071 ms	ArcanistXHPASTLinterTestCase::testLinter
	606 ms	ArcanistCSSLintLinterTestCase::testLinter
	120 ms	ArcanistCSSLintLinterTestCase::testVersion
	130 ms	ArcanistChmodLinterTestCase::testLinter
	587 ms	ArcanistClosureLinterTestCase::testLinter
		View Full Test Results (1 Failed · 46 Passed · 4 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

108 lines

Diff 27758

src/lint/linter/ArcanistXHPASTLinter.php

Loading...