[buddypress-trac] [BuddyPress Trac] #7926: Automated linting for PHP versions

[buddypress-trac]

#7926: Automated linting for PHP versions
-----------------------------------+---------------------
 Reporter:  boonebgorges           |       Owner:  netweb
     Type:  enhancement            |      Status:  new
 Priority:  normal                 |   Milestone:  4.0
Component:  Build/Test Tools       |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  2nd-opinion has-patch  |
-----------------------------------+---------------------
Changes (by boonebgorges):

 * keywords:  needs-patch 2nd-opinion => 2nd-opinion has-patch
 * milestone:  Awaiting Contributions => 4.0


Comment:

 [attachment:"7926.diff"] is a first pass at a linting script. It doesn't
 currently have any exclusions, but the infrastructure is there for it. It
 runs across all PHP files, including tests.

 Note that this kind of fix doesn't actually catch the kind of
 incompatibility discussed in #7925. PHP's linter doesn't execute files to
 test for failures, it only scans for improper syntax. Something like #7925
 could perhaps be caught with a PHPUnit test, or with static code analysis.

 That being said, I think this kind of linting still have value. Any
 thoughts?

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7926#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac