[buddypress-trac] [BuddyPress Trac] #8124: Reintroduce PHP linting

buddypress-trac noreply at wordpress.org
Thu Aug 8 19:41:25 UTC 2019


#8124: Reintroduce PHP linting
------------------------------+--------------------
 Reporter:  boonebgorges      |      Owner:  netweb
     Type:  enhancement       |     Status:  new
 Priority:  normal            |  Milestone:  5.0.0
Component:  Build/Test Tools  |    Version:
 Severity:  normal            |   Keywords:
------------------------------+--------------------
 From @jrf in https://buddypress.trac.wordpress.org/ticket/7997#comment:8:

   While I - of course - fully support adding PHPCompatibilityWP to the QA
 toolset, it is not a replacement for linting code.
   When PHPCompatibility encounters parse errors which are not typically to
 do with PHP cross-version compatibility, it will ignore them and not
 notify you of these.
   Also, in case of parse errors, the results of PHPCompatibility may not
 be reliable as the file may be tokenized incorrectly by PHP/PHPCS because
 of the parse errors.
   I'd strongly recommend using php -l against at least the lowest and
 highest PHP version, if not lowest/highest of each PHP major in addition
 to using PHPCompatibility.

 Let's reintroduce.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8124>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list