Page MenuHomePhabricator
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)
Fri, May 3, 3:59 AM
Unknown Object (File)
Mon, Apr 29, 4:10 PM
Unknown Object (File)
Mon, Apr 29, 12:31 AM
Unknown Object (File)
Sun, Apr 28, 9:10 PM
Unknown Object (File)
Sun, Apr 28, 5:36 AM
Unknown Object (File)
Sat, Apr 27, 11:39 PM
Unknown Object (File)
Wed, Apr 24, 11:56 PM
Unknown Object (File)
Fri, Apr 19, 2:14 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

WIP. Depends on D12295.

Test Plan

WIP

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4115
Build 4128: [Placeholder Plan] Wait for 30 Seconds

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
epriestley requested changes to this revision.Jan 26 2015, 3:14 PM
epriestley added a reviewer: epriestley.

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
joshuaspence updated this revision to Diff 27758.Jan 28 2015, 11:15 AM
joshuaspence edited edge metadata.

Somewhat working

Herald added a subscriber: Korvin. · View Herald TranscriptJan 28 2015, 11:15 AM
joshuaspence updated this revision to Diff 27759.Jan 28 2015, 11:48 AM
joshuaspence edited edge metadata.

Somewhat working

joshuaspence updated this revision to Diff 28041.Feb 3 2015, 7:50 PM

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
joshuaspence updated this revision to Diff 30205.Apr 27 2015, 9:29 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.
joshuaspence updated this revision to Diff 31635.Jun 1 2015, 9:38 PM

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

joshuaspence abandoned this revision.May 17 2019, 1:09 AM

Not actively working on this.

Revision Contents
Changeset List

Diff 27759

View Options

src/lint/linter/ArcanistXHPASTLinter.php

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

src/lint/linter/__tests__/xhpast/reused-iterator-reference.lint-test

Loading...
View Options

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

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility