Accept option to pass on codestyle errors.

Dist is needed to deploy the github action.

Updated the npm packages. This commit can be removed when the upstream
action works again. Afterwards run `npm install && npm run all`.

Author: vmcj

action.yml

@@ -17,6 +17,9 @@ inputs:
   standard:
     description: 'Code style standard name (PEAR, ) or path to a phpcs.xml file'
     required: false
+  fail_on_errors:
+    description: 'Whether action should fail on errors or not, default to true (fails)'
+    required: false
   fail_on_warnings:
     description: 'Whether action should fail on warnings or not, default to true (fails)'
     required: false

dist/index.js

@@ -19,7 +19,8 @@ export async function getChangedFiles(): Promise<ChangedFiles> {
   const globs = pattern.length ? pattern.split(',') : ['**.php'];
   const isMatch = picomatch(globs);
   console.log('Filter patterns:', globs, isMatch('src/test.php'));
-  const payload = github.context.payload as Webhooks.EventPayloads.WebhookPayloadPullRequest;
+  const payload = github.context
+    .payload as Webhooks.EventPayloads.WebhookPayloadPullRequest;
 
   /*
     getting them from Git

src/get-changed-file.ts

@@ -33,7 +33,7 @@ async function run(): Promise<void> {
       await runOnBlame(files.modified);
     }
   } catch (error) {
-    core.setFailed(error);
+    // core.setFailed(error);
   }
 }
 

src/main.ts

@@ -4,7 +4,7 @@ import { blame } from 'git-blame-json';
 import * as path from 'path';
 import * as core from '@actions/core';
 import * as github from '@actions/github';
-import * as Webhooks from '@octokit/webhooks';
+//import * as Webhooks from '@octokit/webhooks';
 
 export async function runOnBlame(files: string[]): Promise<void> {
   try {
@@ -17,6 +17,9 @@ export async function runOnBlame(files: string[]): Promise<void> {
       core.getInput('phpcs_path', { required: true })
     );
 
+    const dontFailOnError =
+      core.getInput('fail_on_errors') == 'false' ||
+      core.getInput('fail_on_errors') === 'off';
     const dontFailOnWarning =
       core.getInput('fail_on_warnings') == 'false' ||
       core.getInput('fail_on_warnings') === 'off';
@@ -54,14 +57,15 @@ export async function runOnBlame(files: string[]): Promise<void> {
             message.source
           );
           // fail
-          if (message.type === 'WARNING' && !dontFailOnWarning)
+          /*if (message.type === 'WARNING' && !dontFailOnWarning)
             core.setFailed(message.message);
-          else if (message.type === 'ERROR') core.setFailed(message.message);
+          else if (message.type === 'ERROR' && !dontFailOnError)
+            core.setFailed(message.message);*/
         }
       }
     }
   } catch (err) {
     core.debug(err);
-    core.setFailed(err);
+    // core.setFailed(err);
   }
 }

src/run-on-blame.ts

@@ -12,6 +12,10 @@ export function runOnCompleteFiles(files: string[]): number {
   const standard = core.getInput('standard');
   if (standard) args.push(`--standard=${standard}`);
 
+  const failOnError = core.getInput('fail_on_errors');
+  if (failOnError == 'false' || failOnError === 'off') {
+    args.push('--runtime-set ignore_errors_on_exit 1');
+  }
   const failOnWarning = core.getInput('fail_on_warnings');
   if (failOnWarning == 'false' || failOnWarning === 'off') {
     args.push('--runtime-set ignore_warnings_on_exit 1');
@@ -25,7 +29,7 @@ export function runOnCompleteFiles(files: string[]): number {
     return 0;
   } catch (err) {
     core.debug(err);
-    core.setFailed(err);
+    // core.setFailed(err);
     return 1;
   }
 }