[#44498] - [5.3] add composer scripts for cs check and add excludes to php-cs-fixer config
- Pending
- Medium
- Build: 5.3-dev
- # 44498
- Diff
- LadySolveig:5.3/dev-tools/composer-scripts-cs
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Add commands
composer run phpcscomposer run csfixer
to composer.json for running cs check same way as it is configured in our automated testing setup, to make it easy for contributors to run it locally.
Add excludes for cacheand logsdirectory to php-cs-fixer configuration as we have the same exlcudes for phpcs.
Improvement for our nice contributors.
Testing Instructions
Open a terminal (and step into the cloned joomla developement folder)
- If not already done: type
composer i - type
composer run phpcs - type
composer run csfixer
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org: joomla/Manual#337
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | External Library Composer Change |
No, absolutely not an expected result.
I have tried to replicate your issue but unfortunately I can't recreate it at the moment.
I have seen you run it directly on Windows.
Since I usually use Linux, I have now installed php 8.3 and composer directly on Windows.
But for me it runs without errors here too.
composer run phpcs
composer run csfixer
Is your IDE or your Git client possibly doing something with the line endings?
is it expected to get this error at the end of running phpcs?
and this error at the end of running csfixer