Skip to content
This repository has been archived by the owner on Aug 20, 2024. It is now read-only.

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Nicolas Vannieuwkerke <[email protected]>
  • Loading branch information
adamrtalbot and nvnieuwk authored Aug 1, 2024
1 parent 85057af commit 5a99d96
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ public class FilePathValidator implements FormatValidator {
@Override
public Optional<String> validate(final String subject) {
if (subject.matches("(s3://|az://|gs://).*")) {
log.debug("S3 paths are not supported by 'FilePathValidator': '${subject}'")
log.debug("Cloud storage paths are not supported by 'FilePathValidator': '${subject}'")
return Optional.empty()
}
Path file = Nextflow.file(subject) as Path
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ public class PathValidator implements FormatValidator {
@Override
public Optional<String> validate(final String subject) {
if (subject.matches("(s3://|az://|gs://).*")) {
log.debug("S3 paths are not supported by 'PathValidator': '${subject}'")
log.debug("Cloud storage paths are not supported by 'PathValidator': '${subject}'")
return Optional.empty()
}
Path file = Nextflow.file(subject) as Path
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -616,7 +616,7 @@ class SchemaValidator extends PluginExtensionPoint {
//
List pathExists(String path, String paramName, Boolean s3PathCheck) {
if (path.matches("(s3://|az://|gs://).*") && !s3PathCheck) {
log.debug "Ignoring validation of S3 URL path '${path}'".toString()
log.debug "Ignoring validation of cloud storage path '${path}'".toString()
} else {
def Path file = Nextflow.file(path) as Path
if (!file.exists()) {
Expand Down

0 comments on commit 5a99d96

Please sign in to comment.