The current parallel-lint command which is run in composer for Codesniffer needs to be modified to also run for .inc files - currently it seems to only check .php files.
Description
Description
Details
Details
| Project | Branch | Lines +/- | Subject | |
|---|---|---|---|---|
| mediawiki/tools/codesniffer | master | +1 -1 | Run phplint against inc files too |
Related Objects
Related Objects
- Mentioned Here
- T92751: Improve test suite for mediawiki phpcs standard
Event Timeline
Comment Actions
Do we have .inc files in the repository? We can add -e php,inc to the command to make it include that file extension.
Comment Actions
This is in relation to T92751 where I was using .inc files. I was unsure how to add .inc files to the composer test.
It may be a good idea to simply add it, even if currently it is not there in the repository.
Comment Actions
Change 261913 had a related patch set uploaded (by Paladox):
Run phplint against inc files too
Comment Actions
Change 261913 abandoned by Hashar:
Run phplint against inc files too
Reason:
mediawiki/tools/codesniffer has no PHP files with .inc .php5 or .phtml. That has been borrowed from mediawiki/core and is of no use here.