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,7 @@ 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

@@ -37,4 +37,4 @@ async function run(): Promise<void> {
   }
 }
 
-run();
+void run();

src/run-on-blame.ts

@@ -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',
@@ -58,7 +59,8 @@ export async function runOnBlame(files: string[]): Promise<void> {
           // fail
           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);
         }
       }
     }

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');