Merge pull request #148 from palantirnet/coder-8-3-7

Use extension list instead of ignore for phpcs

defaults.yml

@@ -230,7 +230,14 @@ phpcs:
 
   # Comma-separated list of patterns for files and directories to exclude from the
   # PHP_CodeSniffer review.
+  #
+  # This is deprecated and will be removed in 3.0, and build.xml will need to be updated then to
+  # use the extensions option (below). This option should not be used with Coder >= 8.3.7, which only
+  # checks php, inc, css, and js by default.
   ignore: "*.md"
+
+  # Comma-separated list of extensions to check in the PHP_CodeSniffer review.
+  extensions: "php,module,inc,install,test,profile,theme,css,info,txt,yml,js"
 
 
 # Configuration for using PHP Mess Detector to check for general PHP best practices,

defaults/install/build.xml

@@ -132,7 +132,7 @@
         </phplint>
 
         <!-- Run PHP Code Sniffer. -->
-        <property name="phpcs.command" value="vendor/bin/phpcs --standard=${phpcs.standard} --ignore=${phpcs.ignore} ${phpcs.directories}" />
+        <property name="phpcs.command" value="vendor/bin/phpcs --standard=${phpcs.standard} --extensions=${phpcs.extensions} ${phpcs.directories}" />
         <echo msg="$> ${phpcs.command}" />
         <exec command="${phpcs.command}" logoutput="true" checkreturn="true" />