magento-lts
magento-lts copied to clipboard
Chore: Add Captainhook & PHPCS to composer require-dev and sample captainhook config
Add captainhook and PHPCS to composer require-dev section Add sample captainhook config
closes #1120 (duplication) refs #947 #982
Please consider, that commit hooks are disabled in captainhook.json
You could test it by installing packages with composer and using the PHPCS Rule with using the command in captainhook.json
I would love to have php-cs fixer in place. This change however is missing few pieces:
- php-cs-fixer configuration file (lets start with config which is not making too much changes in the codebase - and add new rules later on
- github action to validate codestyle on PR
- use friendsofphp/php-cs-fixer v3 instead of v2
Feel free to update my PR
Seems related to #2400