Checking Code Style¶
Before opening a pull request, you should check your code style. If you are using the Vagrant, you can run phpcs within the Drupal installation directory (on the Vagrant, that is /var/www/html/drupal) or from within the web directory (/var/www/html/drupal/web) as follows:
- from within Drupal's root directory:
./vendor/bin/phpcs --standard=./vendor/drupal/coder/coder_sniffer/Drupal modules/contrib/my_module, wheremodules/contrib/my_moduleis the relative or full path to the PHP file you want to check. - from within Drupal's
webdirectory:../vendor/bin/phpcs --standard=../vendor/drupal/coder/coder_sniffer/Drupal yourfile, whereyourfileis the relative or full path to the PHP file you want to check.
In both cases:
- the path to the coding standard file can be relative to where you are running it from, e.g. when in
web:--standard=../vendor/drupal/coder/coder_sniffer/Drupal - you can specify a single file to check, or a directory path; in the latter case, all files in that directory will be checked.
Islandora 8 runs phpcs in its Travis continuous integration environment, and there, it specifies which files to ignore and which files to check. It is a good idea for developers to specify the same options when running phpcs locally, prior to opening a pull request. For example (running phpcs from the within Drupal's web directory), you should use the following --ignore and --extensions options:
../vendor/bin/phpcs --standard=../vendor/drupal/coder/coder_sniffer/Drupal --ignore=*.md --extensions=php,module,inc,install,test,profile,theme,css,info modules/contrib/my_module