Conversation
…t related configurations
|
I checked the This is the reason we had The In conclusion: looks like we might not need that update as phpcsstandards took over the squizlabs package on packagist. Might also need more research how it's intended to be used now |
|
This is quite the fun roundtrip situation. So the new repository https://github.com/PHPCSStandards/PHP_CodeSniffer - actually provides the code for the old package. I've adjusted to revert the name and only update the content-hash to match the new ones. |
|
Yeah that was fun. From my side everything looks good |
The History:
Hope that helps clear things up. |
|
No worries, thank you for the clarification @jrfnl - I appreciate that taking over this project must be quite the endeavor. |
|
Thanks a lot for the write up @jrfnl! |
This pull request updates the project to use the
PHPCSStandards/PHP_CodeSnifferpackage instead ofsquizlabs/PHP_CodeSniffer, and makes related adjustments throughout the codebase. It also updates the Docker image reference and adds initial support for PHP 8.4 in the test matrix.This closes #34
Dependency and documentation updates:
squizlabs/php_codesniffertophpcsstandards/php_codesnifferincomposer.json.README.mdfromsquizlabs/PHP_CodeSniffertoPHPCSStandards/PHP_CodeSniffer, including links and package names. [1] [2]Build and testing changes:
Makefileto useflyeralarm/php-code-validatorinstead offlyeralarm/php-code-valiator-cli.Makefile'stest-alltarget.