Removes debug statements, which shouldn't be in production ever
Fixers set for PHP-CS-Fixer. Removes debug statements, which shouldn't be in production ever.
If you have ever had 500 Error on production because
dump() function is missing,
or you forget to remove debug statements time to time - this small package is for you.
Debug statements are good for debugging, but it should never get to master branch.
Initially proposed as RFC in PHP-CS-Fixer repository, it was considered too risky to have it in core (see discussion at https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2218)
N.B. These fixers are risky and potentially can break your application. You should understand consequences of having it in your project (especially a legacy one). You are warned.
$ composer require drew/debug-statements-fixers:^0.4 --dev
Adjust your PHP-CS-Fixer config:
# .php_cs.dist <?php $finder = PhpCsFixer\Finder::create() ->in([__DIR__.'/src', __DIR__.'/tests']); return PhpCsFixer\Config::create() ->setRules([ //any fixers you would like to have 'RemoveDebugStatements/dump' => true, ]) ->registerCustomFixers([new Drew\DebugStatementsFixers\Dump()]) ->setRiskyAllowed(true) ->setFinder($finder);
Works best when integrated with your CI server, just add this step to your CI config:
$ php vendor/bin/php-cs-fixer fix --diff --dry-run -v