PHP Pre-commit Hooks

Pre-commit scripts appropriate for PHP project. These hooks are made as custom plugins under the pre-commit git hook framework.

Setup

Just add to your .pre-commit-config.yaml file with the following

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  rev: master
  hooks:
  - id: php-lint
  - id: php-unit
  - id: php-cs
    files: \.(php)$
    args: ["--standard=PSR2,EcgM2", "--severity=10", "--extensions=php,phtml", "-p"]
  - id: php-md
    files: \.(php)$
    args: ["codesize,controversial,design,naming,unusedcode"]
  - id: php-no-dumps
  - id: php-no-var_dumps
  - id: php-no-exits
  - id: php-stan
    files: \.(php)$
    args: ["--configuration=phpstan.neon --level=5"]

Supported Hooks

php-lint

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-lint

A bash script that runs php -l against stage files that are php. Assumes php is a global executable command. Will exit when it hits the first syntax error.

php-lint-all

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-lint-all

A systems hook that just runs php -l against stage files that have the .php extension. Add the args: [-s first] in your .pre-commit-config.yaml to enable it to exit on the first error found.

php-unit

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-unit

A bash script that will run the appropriate phpunit executable. It will assume

• Find the executable to run at either vendor/bin/phpunit, phpunit or php phpunit.phar (in that exact order).
• There is already a phpunit.xml in the root of the repo

Note in its current state, it will run the whole PHPUnit test as along as .php file was committed.

php-cs

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-cs
    files: \.(php)$
    args: [--standard=PSR1 -p]

Similar pattern as the php-unit hook. A bash script that will run the appropriate PHP Code Sniffer executable.

It will assume that there is a valid PHP Code Sniffer executable at these locations, vendor/bin/phpcs, phpcs or php phpcs.phar (in that exact order).

The args property in your hook declaration can be used for pass any valid PHP Code Sniffer arguments. In the example above, it will run PHP Code Sniffer against only the staged php files with the PSR-1 and progress enabled.

If you have multiple standards or a comma in your args property, escape the comma character like so

php-cbf

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-cs
    files: \.(php)$
    args: [--standard=PSR1 -p]

Similar pattern as the php-cs hook. A bash script that will run the appropriate PHP Code Sniffer executable and will try to fix errors if it can using phpcbf.

It will assume that there is a valid PHP Code Beautifier and Fixer executable at these locations, vendor/bin/phpcbf, phpcbf or php phpcbf.phar (in that exact order).

The args property in your hook declaration can be used for pass any valid PHP Code Sniffer arguments. In the example above, it will run PHP Code Sniffer against only the staged php files with the PSR-1 and progress enabled.

If you have multiple standards or a comma in your args property, escape the comma character like so

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-cs
    files: \.(php)$
    args: ["--standard=PSR1/,path/to/ruleset.xml", "-p"]

To install PHP Codesniffer (phpcs & phpcbf), follow the recommended steps here.

php-cs-fixer

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-cs-fixer
    files: \.(php)$
    args: [--level=PSR2]

Similar pattern as the php-cs hook. A bash script that will run the appropriate PHP Coding Standards Fixer executable and to fix errors according to the configuration. It accepts all of the args from the php-cs-fixer command, in particular the --level, --config, and --config-file options.

The tool will fail a build when it has made changes to the staged files. This allows a developer to do a git diff and examine the changes that it has made. Remember that you may omit this if needed with a SKIP=php-cs-fixer git commit.

php-no-dumps

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-no-dumps

A regex that checks that the symfony dump debug function isn't committed

php-no-var_dumps

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-no-var_dumps

A regex that checks that var_dump isn't committed

php-no-exits

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-no-exits

A regex that checks that exit and die aren't committed

php-md

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-md
    files: \.(php)$
    args: ["codesize,controversial,design,naming,unusedcode"]

A bash script that will run the appropriate PHP Mess Detector executable and report issues as configured.

The tool will fail a build when it has found issues that violate the configured code rules. Please note that the code rule list must be the first argument in the args list.

php-stan (PHP7 only)

PHPStan focuses on finding errors in your code without actually running it. It catches whole classes of bugs even before you write tests for the code.

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-stan
    files: \.(php)$
    args: ["--configuration=phpstan.neon --level=5"]

A bash script that will run the appropriate phpstan executable.

php-cpd

- repo: git@github.com:justin-at-bounteous/pre-commit-php.git
  hooks:
  - id: php-cpd
    files: \.(php)$
    args: ["--min-tokens=10"]

A bash script that will run the appropriate PHP Copy Paste Detector executable and report on duplicate code.

The tool will fail a build when it has found issues that violate the configured code rules. This will accept all arguments, in particular you'll want to tune for ----min-lines=, --min-tokens=, and --fuzzy.