Skip to content

Commit 5afb421

Browse files
committed
Move PathUtils into nf-lang
Signed-off-by: Ben Sherman <[email protected]>
1 parent f89c0db commit 5afb421

File tree

4 files changed

+7
-104
lines changed

4 files changed

+7
-104
lines changed

src/main/java/nextflow/lsp/NextflowLanguageServer.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
import java.util.Optional;
2626
import java.util.concurrent.CompletableFuture;
2727

28-
import nextflow.lsp.file.PathUtils;
2928
import nextflow.lsp.util.JsonUtils;
3029
import nextflow.lsp.util.Logger;
3130
import nextflow.lsp.util.ProgressNotification;
@@ -36,6 +35,7 @@
3635
import nextflow.lsp.services.config.ConfigService;
3736
import nextflow.lsp.services.script.ScriptService;
3837
import nextflow.script.formatter.FormattingOptions;
38+
import nextflow.util.PathUtils;
3939
import org.codehaus.groovy.runtime.DefaultGroovyMethods;
4040
import org.eclipse.lsp4j.CallHierarchyIncomingCall;
4141
import org.eclipse.lsp4j.CallHierarchyIncomingCallsParams;
@@ -602,7 +602,7 @@ private LanguageService getLanguageService(String uri) {
602602
return null;
603603
}
604604
var path = Path.of(URI.create(uri));
605-
if( PathUtils.isPathExcluded(path, configuration.excludePatterns()) )
605+
if( PathUtils.isExcluded(path, configuration.excludePatterns()) )
606606
return null;
607607
return service;
608608
}

src/main/java/nextflow/lsp/file/PathUtils.java

-37
This file was deleted.

src/main/java/nextflow/lsp/services/LanguageService.java

+5-17
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,13 @@
3535

3636
import nextflow.lsp.ast.ASTNodeCache;
3737
import nextflow.lsp.file.FileCache;
38-
import nextflow.lsp.file.PathUtils;
3938
import nextflow.lsp.util.DebouncingExecutor;
4039
import nextflow.lsp.util.LanguageServerUtils;
4140
import nextflow.lsp.util.Logger;
4241
import nextflow.lsp.util.Positions;
4342
import nextflow.script.control.RelatedInformationAware;
4443
import nextflow.script.formatter.FormattingOptions;
44+
import nextflow.util.PathUtils;
4545
import org.codehaus.groovy.runtime.DefaultGroovyMethods;
4646
import org.eclipse.lsp4j.CallHierarchyIncomingCall;
4747
import org.eclipse.lsp4j.CallHierarchyItem;
@@ -143,23 +143,11 @@ public void initialize(String rootUri, LanguageServerConfiguration configuration
143143

144144
protected Set<URI> getWorkspaceFiles(String rootUri) {
145145
try {
146-
var root = Path.of(URI.create(rootUri));
147146
var result = new HashSet<URI>();
148-
Files.walkFileTree(root, new SimpleFileVisitor<Path>() {
149-
@Override
150-
public FileVisitResult preVisitDirectory(Path path, BasicFileAttributes attrs) {
151-
return PathUtils.isPathExcluded(path, configuration.excludePatterns())
152-
? FileVisitResult.SKIP_SUBTREE
153-
: FileVisitResult.CONTINUE;
154-
}
155-
@Override
156-
public FileVisitResult visitFile(Path path, BasicFileAttributes attrs) {
157-
if( matchesFile(path.toString()) && !PathUtils.isPathExcluded(path, configuration.excludePatterns()) )
158-
result.add(path.toUri());
159-
return FileVisitResult.CONTINUE;
160-
}
161-
});
162-
147+
PathUtils.visitFiles(
148+
Path.of(URI.create(rootUri)),
149+
(path) -> (Files.isDirectory(path) || matchesFile(path.toString())) && !PathUtils.isExcluded(path, configuration.excludePatterns()),
150+
(path) -> result.add(path.toUri()));
163151
return result;
164152
}
165153
catch( IOException e ) {

src/test/groovy/nextflow/lsp/file/PathUtilsTest.groovy

-48
This file was deleted.

0 commit comments

Comments
 (0)