[wp-trac] [WordPress Trac] #49722: Don't format before linting on Travis

WordPress Trac noreply at wordpress.org
Sat Mar 28 20:33:47 UTC 2020


#49722: Don't format before linting on Travis
------------------------------+-----------------------------
 Reporter:  johnbillion       |      Owner:  (none)
     Type:  defect (bug)      |     Status:  new
 Priority:  normal            |  Milestone:  Awaiting Review
Component:  Build/Test Tools  |    Version:
 Severity:  normal            |   Keywords:  dev-feedback
  Focuses:                    |
------------------------------+-----------------------------
 When the `lint:php` job runs on Travis, the code is formatted (with
 phpcbf) before it's linted. This defeats the point of linting as it means
 problems that can be fixed with phpcbf get missed.

 Ref:
 https://core.trac.wordpress.org/browser/tags/5.3.2/.travis.yml?marks=124-125#L110

 Introduced in [45799]. @pento Is this intentional or was it something
 temporary that was supposed to be removed at a later date?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/49722>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list