Page MenuHomePhabricator

Adding php -l linter
ClosedPublic

Authored by bluehawk on Aug 27 2014, 9:15 PM.
Tags
None
Referenced Files
F15504807: D10370.id24968.diff
Mon, Apr 14, 8:53 PM
F15499995: D10370.id.diff
Sun, Apr 13, 3:46 PM
F15499918: D10370.id24973.diff
Sun, Apr 13, 3:30 PM
F15499917: D10370.id24968.diff
Sun, Apr 13, 3:30 PM
F15499916: D10370.id24960.diff
Sun, Apr 13, 3:29 PM
F15499866: D10370.diff
Sun, Apr 13, 3:19 PM
F15491255: D10370.id24973.diff
Sat, Apr 12, 12:56 AM
F15478105: D10370.id24968.diff
Mon, Apr 7, 8:35 PM

Details

Summary

Adds a linter that checks php files for syntax errors using php -l

Test Plan

Add a section to your .arclint file similar to:

"php": {
  "type": "php",
  "include": "(\\.php$)"
}

Then run arc lint on a php file with a syntax error.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

bluehawk retitled this revision from to Adding php -l linter.
bluehawk updated this object.
bluehawk edited the test plan for this revision. (Show Details)
bluehawk added a reviewer: epriestley.
bluehawk added a subscriber: jacobwalker0814.
epriestley edited edge metadata.

Add a unit test too?

src/lint/linter/ArcanistPhpLinter.php
19

Add punctuation.

35

Add punctuation.

39

Return 'php' instead. Don't introduce deprecated configuration. The "bin" option covers this now.

This revision now requires changes to proceed.Aug 27 2014, 9:20 PM
bluehawk edited edge metadata.

Cleanup, more robust, and adding test

This revision is now accepted and ready to land.Aug 28 2014, 12:29 AM
epriestley updated this revision to Diff 24973.

Closed by commit rARC9fc8a2f61b19 (authored by @bluehawk, committed by @epriestley).