more logic around danger

AbhiPrasad · AbhiPrasad · commit d2ba5fc83818 · 2020-08-03T09:03:47.000-04:00
diff --git a/dangerfile.ts b/dangerfile.ts
@@ -16,7 +16,7 @@ async function eslint(): Promise<void[]> {
   const allFiles = danger.git.created_files.concat(danger.git.modified_files);
   const cli = new CLIEngine({});
   // let eslint filter down to non-ignored, matching the extensions expected
-  const filesToLint = allFiles.filter(f => !cli.isPathIgnored(f));
+  const filesToLint = allFiles.filter(f => !cli.isPathIgnored(f) || !f.endsWith('lock') || !f.endsWith('json'));
   return Promise.all(filesToLint.map(f => lintFile(cli, f)));
 }
 

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ async function eslint(): Promise<void[]> {`
`16`	`16`	`const allFiles = danger.git.created_files.concat(danger.git.modified_files);`
`17`	`17`	`const cli = new CLIEngine({});`
`18`	`18`	`// let eslint filter down to non-ignored, matching the extensions expected`
`19`		`- const filesToLint = allFiles.filter(f => !cli.isPathIgnored(f));`
	`19`	`+ const filesToLint = allFiles.filter(f => !cli.isPathIgnored(f) \|\| !f.endsWith('lock') \|\| !f.endsWith('json'));`
`20`	`20`	`return Promise.all(filesToLint.map(f => lintFile(cli, f)));`
`21`	`21`	`}`
`22`	`22`