Host a working version

.gitignore

@@ -96,4 +96,5 @@ Thumbs.db
 
 # Ignore built ts files
 __tests__/runner/*
-lib/**/*
+lib/**/*
+

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

src/get-changed-file.ts

@@ -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.WebhookPayloadPullRequest;
+  const payload = github.context
+    .payload as Webhooks.EventPayloads.WebhookPayloadPullRequest;
 
   /*
     getting them from Git

src/main.ts

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

src/run-on-blame.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';
@@ -26,8 +29,6 @@ export async function runOnBlame(files: string[]): Promise<void> {
     }
 
     // blame files and output relevant errors
-    const payload = github.context
-      .payload as Webhooks.WebhookPayloadPullRequest;
     // get email of author of first commit in PR
     const authorEmail = execFileSync(
       'git',
@@ -56,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-files.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;
   }
 }