Page MenuHomePhabricator
Differential D11265

Minor improvements to `ArcanistPhpLinter`
ClosedPublic
Actions

Authored by joshuaspence on Jan 7 2015, 9:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 6:12 PM
Unknown Object (File)
Fri, Dec 20, 5:44 PM
Unknown Object (File)
Tue, Dec 17, 10:51 AM
Unknown Object (File)
Tue, Dec 17, 5:15 AM
Unknown Object (File)
Fri, Dec 13, 12:35 AM
Unknown Object (File)
Thu, Dec 12, 10:40 PM
Unknown Object (File)
Thu, Dec 5, 1:19 AM
Unknown Object (File)
Nov 11 2024, 7:27 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC5492f0fad6c6: Minor improvements to `ArcanistPhpLinter`
Summary

Various minor improvements to ArcanistPhpLinter.

Test Plan

arc unit

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 27064.Jan 7 2015, 9:29 AM
joshuaspence retitled this revision from to Minor improvements to `ArcanistPhpLinter`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 7 2015, 9:29 AM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley accepted this revision.Jan 7 2015, 12:54 PM
epriestley added a reviewer: epriestley.
This revision is now accepted and ready to land.Jan 7 2015, 12:54 PM
epriestley requested changes to this revision.Jan 7 2015, 12:54 PM
epriestley edited edge metadata.

Oh, WIP + test failure.

This revision now requires changes to proceed.Jan 7 2015, 12:54 PM
epriestley added a comment.Jan 7 2015, 12:56 PM

FWIW, I do not get "PHP" in the output in 5.4.30:

epriestley@orbital ~/dev/phabricator $ php -v
PHP 5.4.30 (cli) (built: Jul 29 2014 23:43:29) 
Copyright (c) 1997-2014 The PHP Group
Zend Engine v2.4.0, Copyright (c) 1998-2014 Zend Technologies
epriestley@orbital ~/dev/phabricator $ php -l derp 

Parse error: parse error in derp on line 3
Errors parsing derp

Note "Parse error", not "PHP Parse error".

joshuaspence updated this revision to Diff 27090.Jan 8 2015, 8:31 AM
joshuaspence edited edge metadata.

Fix

Herald added a subscriber: Korvin. · View Herald TranscriptJan 8 2015, 8:31 AM
joshuaspence updated this revision to Diff 27091.Jan 8 2015, 8:34 AM
joshuaspence edited edge metadata.

s/php/PHP/

joshuaspence updated this revision to Diff 27092.Jan 8 2015, 8:36 AM

Don't allow linter severity to be customized

joshuaspence updated this object.Jan 8 2015, 8:37 AM
joshuaspence edited the test plan for this revision. (Show Details)
epriestley accepted this revision.Jan 8 2015, 8:38 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 8 2015, 8:38 AM
epriestley added inline comments.Jan 8 2015, 8:38 AM
src/lint/linter/ArcanistPhpLinter.php
97

Maybe phutil_utf8_strtolower().

joshuaspence updated this revision to Diff 27094.Jan 8 2015, 8:39 AM
joshuaspence edited edge metadata.

Use getLintMessageName

Closed by commit rARC5492f0fad6c6: Minor improvements to `ArcanistPhpLinter` (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 8 2015, 8:41 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
lint/
linter/
60 lines

Diff 27095

View Options

src/lint/linter/ArcanistPhpLinter.php

Loading...
Phabricator · Built by Phacility