[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