filter ** paths

src/analysis-paths.test.ts

@@ -16,10 +16,10 @@ test("emptyPaths", async t => {
 
 test("nonEmptyPaths", async t => {
   let config = new configUtils.Config();
-  config.paths.push('path1', 'path2');
-  config.pathsIgnore.push('path3', 'path4');
+  config.paths.push('path1', 'path2', '**/path3');
+  config.pathsIgnore.push('path4', 'path5', 'path6/**');
   analysisPaths.includeAndExcludeAnalysisPaths(config, []);
   t.is(process.env['LGTM_INDEX_INCLUDE'], 'path1\npath2');
-  t.is(process.env['LGTM_INDEX_EXCLUDE'], 'path3\npath4');
-  t.is(process.env['LGTM_INDEX_FILTERS'], 'include:path1\ninclude:path2\nexclude:path3\nexclude:path4');
+  t.is(process.env['LGTM_INDEX_EXCLUDE'], 'path4\npath5');
+  t.is(process.env['LGTM_INDEX_FILTERS'], 'include:path1\ninclude:path2\ninclude:**/path3\nexclude:path4\nexclude:path5\nexclude:path6/**');
 });

src/analysis-paths.ts

@@ -6,18 +6,24 @@ function isInterpretedLanguage(language): boolean {
   return language === 'javascript' || language === 'python';
 }
 
+// Builds an environment variable suitable for LGTM_INDEX_INCLUDE or LGTM_INDEX_EXCLUDE
+function buildIncludeExcludeEnvVar(paths: string[]): string {
+  return paths.filter(p => p.indexOf('**') === -1).join('\n');
+}
+
 export function includeAndExcludeAnalysisPaths(config: configUtils.Config, languages: string[]) {
   // The 'LGTM_INDEX_INCLUDE' and 'LGTM_INDEX_EXCLUDE' environment variables
   // control which files/directories are traversed when scanning.
   // This allows including files that otherwise would not be scanned, or
   // excluding and not traversing entire file subtrees.
   // It does not understand double-globs because that would require it to
   // traverse the entire file tree to determine which files are matched.
+  // Any paths containing "**" are not included in these.
   if (config.paths.length !== 0) {
-    core.exportVariable('LGTM_INDEX_INCLUDE', config.paths.join('\n'));
+    core.exportVariable('LGTM_INDEX_INCLUDE', buildIncludeExcludeEnvVar(config.paths));
   }
   if (config.pathsIgnore.length !== 0) {
-    core.exportVariable('LGTM_INDEX_EXCLUDE', config.pathsIgnore.join('\n'));
+    core.exportVariable('LGTM_INDEX_EXCLUDE', buildIncludeExcludeEnvVar(config.pathsIgnore));
   }
 
   // The 'LGTM_INDEX_FILTERS' environment variable controls which files are