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