[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