PHPCS 3.0 will throw exit code 2 for fixable errors

See squizlabs/PHP_CodeSniffer#930

Author: arnested

index.js

@@ -202,8 +202,8 @@ var phpcsPlugin = function(options) {
                 return;
             }
 
-            if (exitCode > 1) {
-                // On codding style problems Code Sniffer should exists with "1" code.
+            if (exitCode > 2) {
+                // On codding style problems Code Sniffer should exists with "1" or "2" code.
                 // All other non-zero exit codes should be treated as Code Sniffer errors.
                 var phpcsError = new gutil.PluginError('gulp-phpcs', 'Execution of Code Sniffer Failed');
                 phpcsError.stdout = output;
@@ -218,7 +218,7 @@ var phpcsPlugin = function(options) {
                 output: ''
             };
 
-            if (exitCode === 1) {
+            if ((exitCode === 1) || (exitCode === 2)) {
                 // A codding style problem is found. Attache report to the file to allow
                 // reporters do their job.
                 report.error = true;

test/fixture/error

@@ -4,5 +4,5 @@
 cat > /dev/null
 # It's just an arbitrary text.
 echo "This is a test error."
-# PHPCS uses exit-codes greater than 1 to report about real errors.
-exit 2
+# PHPCS uses exit-codes greater than 2 to report about real errors.
+exit 3